diff options
author | Arne Juul <arnej@yahoo-inc.com> | 2018-11-13 09:46:15 +0000 |
---|---|---|
committer | gjoranv <gv@oath.com> | 2019-01-21 15:09:26 +0100 |
commit | 8a292274bd5e3df677d5088acf07c6dad75ee7c8 (patch) | |
tree | e1531f4ed487fc111a1ec6f8242218ca927fc622 /vespaclient-java/src/main/java/com | |
parent | 854dfb8d8eea6b651b96eeb2d4b2b51830e75226 (diff) |
remove or disable --headersonly flag
Diffstat (limited to 'vespaclient-java/src/main/java/com')
-rw-r--r-- | vespaclient-java/src/main/java/com/yahoo/vespaget/CommandLineOptions.java | 12 | ||||
-rw-r--r-- | vespaclient-java/src/main/java/com/yahoo/vespavisit/VdsVisit.java | 4 |
2 files changed, 7 insertions, 9 deletions
diff --git a/vespaclient-java/src/main/java/com/yahoo/vespaget/CommandLineOptions.java b/vespaclient-java/src/main/java/com/yahoo/vespaget/CommandLineOptions.java index 7cc7b95bdae..703cdeba247 100644 --- a/vespaclient-java/src/main/java/com/yahoo/vespaget/CommandLineOptions.java +++ b/vespaclient-java/src/main/java/com/yahoo/vespaget/CommandLineOptions.java @@ -68,7 +68,7 @@ public class CommandLineOptions { options.addOption(Option.builder("e") .hasArg(false) - .desc("Retrieve header fields only. [Deprecated].") + .desc("Retrieve header fields only. [Removed in Vespa 7].") .longOpt(HEADERSONLY_OPTION).build()); options.addOption(Option.builder("f") @@ -180,17 +180,15 @@ public class CommandLineOptions { throw new IllegalArgumentException("Cannot combine both xml and json output"); } - if (printIdsOnly && headersOnly) { - throw new IllegalArgumentException("Print ids and headers only options are mutually exclusive."); + if (headersOnly) { + throw new IllegalArgumentException("Headers only option has been removed."); } - if ((printIdsOnly || headersOnly) && !fieldSet.isEmpty()) { - throw new IllegalArgumentException("Field set option can not be used in combination with print ids or headers only options."); + if (printIdsOnly && !fieldSet.isEmpty()) { + throw new IllegalArgumentException("Field set option can not be used in combination with print ids option."); } if (printIdsOnly) { fieldSet = "[id]"; - } else if (headersOnly) { - fieldSet = "[header]"; } else if (fieldSet.isEmpty()) { fieldSet = "[all]"; } diff --git a/vespaclient-java/src/main/java/com/yahoo/vespavisit/VdsVisit.java b/vespaclient-java/src/main/java/com/yahoo/vespavisit/VdsVisit.java index 0f8f0df0f27..4430af19fbc 100644 --- a/vespaclient-java/src/main/java/com/yahoo/vespavisit/VdsVisit.java +++ b/vespaclient-java/src/main/java/com/yahoo/vespavisit/VdsVisit.java @@ -189,7 +189,7 @@ public class VdsVisit { .desc("Only visit up to the given timestamp (microseconds).") .type(Number.class).build()); - options.addOption("e", "headersonly", false, "Only visit headers of documents.[Deprecated]"); + options.addOption("e", "headersonly", false, "Only visit headers of documents.[Removed in Vespa 7]"); options.addOption(Option.builder("l") .longOpt("fieldset") @@ -480,7 +480,7 @@ public class VdsVisit { params.setToTimestamp(((Number) line.getParsedOptionValue("t")).longValue()); } if (line.hasOption("e")) { - params.fieldSet("[header]"); + throw new IllegalArgumentException("Headers only option has been removed."); } if (line.hasOption("l")) { params.fieldSet(line.getOptionValue("l")); |