Merge pull request #2148 from yahoo/bjorncs/forbid-fast-access-prediate-tensor-reference-attribute

Forbid fast-access for predicate, tensor and reference attributes

2 files changed, 54 insertions, 0 deletions

diff --git a/config-model/src/main/java/com/yahoo/searchdefinition/processing/FastAccessValidator.java b/config-model/src/main/java/com/yahoo/searchdefinition/processing/FastAccessValidator.java
new file mode 100644
index 00000000000..7bef7244eb7
--- /dev/null
+++ b/config-model/src/main/java/com/yahoo/searchdefinition/processing/FastAccessValidator.java
@@ -0,0 +1,53 @@
+// Copyright 2017 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
+package com.yahoo.searchdefinition.processing;
+
+import com.yahoo.config.application.api.DeployLogger;
+import com.yahoo.searchdefinition.RankProfileRegistry;
+import com.yahoo.searchdefinition.Search;
+import com.yahoo.searchdefinition.document.Attribute;
+import com.yahoo.vespa.model.container.search.QueryProfiles;
+
+import java.util.stream.Collectors;
+
+/**
+ * Validates the use of the fast-access property.
+ *
+ * @author bjorncs
+ */
+public class FastAccessValidator extends Processor {
+    public FastAccessValidator(Search search, DeployLogger deployLogger, RankProfileRegistry rankProfileRegistry, QueryProfiles queryProfiles) {
+        super(search, deployLogger, rankProfileRegistry, queryProfiles);
+    }
+
+    @Override
+    public void process() {
+        String invalidAttributes = search.allFields()
+                .flatMap(field -> field.getAttributes().values().stream())
+                .filter(FastAccessValidator::isIncompatibleAttribute)
+                .map(Attribute::getName)
+                .collect(Collectors.joining(", "));
+        if (!invalidAttributes.isEmpty()) {
+            throw new IllegalArgumentException(
+                    String.format(
+                            "For search '%s': The following attributes have a type that is incompatible with fast-access: %s. " +
+                                    "Predicate, tensor and reference attributes are incompatible with fast-access.",
+                            search.getName(),
+                            invalidAttributes));
+        }
+    }
+
+    private static boolean isIncompatibleAttribute(Attribute attribute) {
+        return attribute.isFastAccess() && isTypeIncompatibleWithFastAccess(attribute.getType());
+    }
+
+    private static boolean isTypeIncompatibleWithFastAccess(Attribute.Type type) {
+        switch (type) {
+            case PREDICATE:
+            case TENSOR:
+            case REFERENCE:
+                return true;
+            default:
+                return false;
+        }
+    }
+}
diff --git a/config-model/src/main/java/com/yahoo/searchdefinition/processing/Processing.java b/config-model/src/main/java/com/yahoo/searchdefinition/processing/Processing.java
index 2098d0c0808..18cfd227813 100644
--- a/config-model/src/main/java/com/yahoo/searchdefinition/processing/Processing.java
+++ b/config-model/src/main/java/com/yahoo/searchdefinition/processing/Processing.java
@@ -73,6 +73,7 @@ public class Processing {
                 TensorFieldProcessor::new,
                 RankProfileTypeSettingsProcessor::new,
                 ReferenceFieldsProcessor::new,
+                FastAccessValidator::new,
 
                 // These two should be last.
                 IndexingValidation::new,