diff options
author | Jon Bratseth <bratseth@vespa.ai> | 2023-05-16 12:49:12 +0200 |
---|---|---|
committer | Jon Bratseth <bratseth@vespa.ai> | 2023-05-16 12:49:12 +0200 |
commit | b7d6ca3d0dd94f33ca5793ec9b9d427da2a1623b (patch) | |
tree | 5289154166f6996e6075bce1e7e7e581d467a4a0 | |
parent | 194f2afcd1805f3a0f1bd95672130d96b3d4c314 (diff) |
Classify missing node tag for cloud prod as illegal argument
-rw-r--r-- | config-model/src/main/java/com/yahoo/schema/processing/AttributesImplicitWord.java | 6 | ||||
-rw-r--r-- | configserver/src/main/java/com/yahoo/vespa/config/server/provision/ProvisionerAdapter.java | 4 |
2 files changed, 5 insertions, 5 deletions
diff --git a/config-model/src/main/java/com/yahoo/schema/processing/AttributesImplicitWord.java b/config-model/src/main/java/com/yahoo/schema/processing/AttributesImplicitWord.java index 415f23f2786..f0b673920da 100644 --- a/config-model/src/main/java/com/yahoo/schema/processing/AttributesImplicitWord.java +++ b/config-model/src/main/java/com/yahoo/schema/processing/AttributesImplicitWord.java @@ -44,10 +44,8 @@ public class AttributesImplicitWord extends Processor { private boolean fieldImplicitlyWordMatch(ImmutableSDField field) { // numeric types should not trigger exact-match query parsing - DataType dt = field.getDataType().getPrimitiveType(); - if (dt != null && dt instanceof NumericDataType) { - return false; - } + if (field.getDataType().getPrimitiveType() instanceof NumericDataType) return false; + return (! field.hasIndex() && !field.getAttributes().isEmpty() && field.getIndices().isEmpty() diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/provision/ProvisionerAdapter.java b/configserver/src/main/java/com/yahoo/vespa/config/server/provision/ProvisionerAdapter.java index 29cd44b57a4..4d1f058b005 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/provision/ProvisionerAdapter.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/provision/ProvisionerAdapter.java @@ -35,7 +35,9 @@ public class ProvisionerAdapter implements HostProvisioner { @Override public HostSpec allocateHost(String alias) { // TODO: Remove this method since hosted/non-hosted needs different interfaces. See also ModelContextImpl.getHostProvisioner - throw new UnsupportedOperationException("Clusters in hosted environments must have a <nodes count='N'> tag " + + // Illegal argument becaiuse we end here by following a path not suitable for the environment steered + // by the content of the nodes tag in services + throw new IllegalArgumentException("Clusters in hosted environments must have a <nodes count='N'> tag " + "matching all zones, and having no <node> subtags, " + "see https://cloud.vespa.ai/en/reference/services"); } |