diff options
author | Jon Marius Venstad <venstad@gmail.com> | 2021-04-30 08:27:33 +0200 |
---|---|---|
committer | Jon Marius Venstad <venstad@gmail.com> | 2021-04-30 08:27:33 +0200 |
commit | a77415f2827e3f39edd06439a98698c7a279b9a1 (patch) | |
tree | a249b310ac96a99bf2d8b467caa79eb5219c8119 /config-model/src/main/java/com/yahoo/vespa/model | |
parent | fe6420e4d33cf0e19bd7ff865cc55c728ad8b6aa (diff) |
Add reindex actions when extra fields are added
Diffstat (limited to 'config-model/src/main/java/com/yahoo/vespa/model')
-rw-r--r-- | config-model/src/main/java/com/yahoo/vespa/model/application/validation/change/search/IndexingScriptChangeValidator.java | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/application/validation/change/search/IndexingScriptChangeValidator.java b/config-model/src/main/java/com/yahoo/vespa/model/application/validation/change/search/IndexingScriptChangeValidator.java index 91e370211f1..55cad4629de 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/application/validation/change/search/IndexingScriptChangeValidator.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/application/validation/change/search/IndexingScriptChangeValidator.java @@ -2,7 +2,6 @@ package com.yahoo.vespa.model.application.validation.change.search; import com.yahoo.config.application.api.ValidationId; -import com.yahoo.config.application.api.ValidationOverrides; import com.yahoo.config.provision.ClusterSpec; import com.yahoo.searchdefinition.Search; import com.yahoo.searchdefinition.document.ImmutableSDField; @@ -13,8 +12,8 @@ import com.yahoo.vespa.indexinglanguage.expressions.ScriptExpression; import com.yahoo.vespa.model.application.validation.change.VespaConfigChangeAction; import com.yahoo.vespa.model.application.validation.change.VespaReindexAction; -import java.time.Instant; import java.util.ArrayList; +import java.util.LinkedHashSet; import java.util.List; import java.util.Optional; @@ -43,6 +42,12 @@ public class IndexingScriptChangeValidator { if (currentField != null) { validateScripts(currentField, nextField).ifPresent(r -> result.add(r)); } + else if (nextField.isExtraField()) { + result.add(VespaReindexAction.of(id, + ValidationId.indexingChange, + "Non-document field '" + nextField.getName() + + "' added; this may be populated by reindexing")); + } } return result; } |