aboutsummaryrefslogtreecommitdiffstats
path: root/vespaclient-java/src/main/java/com
diff options
context:
space:
mode:
authorArne Juul <arnej@yahoo-inc.com>2018-11-13 09:46:15 +0000
committergjoranv <gv@oath.com>2019-01-21 15:09:26 +0100
commit8a292274bd5e3df677d5088acf07c6dad75ee7c8 (patch)
treee1531f4ed487fc111a1ec6f8242218ca927fc622 /vespaclient-java/src/main/java/com
parent854dfb8d8eea6b651b96eeb2d4b2b51830e75226 (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.java12
-rw-r--r--vespaclient-java/src/main/java/com/yahoo/vespavisit/VdsVisit.java4
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"));