aboutsummaryrefslogtreecommitdiffstats
path: root/config-model/src/main/java/com/yahoo/searchdefinition/processing/AddExtraFieldsToDocument.java
diff options
context:
space:
mode:
Diffstat (limited to 'config-model/src/main/java/com/yahoo/searchdefinition/processing/AddExtraFieldsToDocument.java')
-rw-r--r--config-model/src/main/java/com/yahoo/searchdefinition/processing/AddExtraFieldsToDocument.java10
1 files changed, 9 insertions, 1 deletions
diff --git a/config-model/src/main/java/com/yahoo/searchdefinition/processing/AddExtraFieldsToDocument.java b/config-model/src/main/java/com/yahoo/searchdefinition/processing/AddExtraFieldsToDocument.java
index 233075155b5..f846eabbff8 100644
--- a/config-model/src/main/java/com/yahoo/searchdefinition/processing/AddExtraFieldsToDocument.java
+++ b/config-model/src/main/java/com/yahoo/searchdefinition/processing/AddExtraFieldsToDocument.java
@@ -25,6 +25,11 @@ public class AddExtraFieldsToDocument extends Processor {
super(search, deployLogger, rankProfileRegistry, queryProfiles);
}
+ //TODO This is a tempoarry hack to avoid producing illegal code for fields not wanted anyway.
+ private boolean dirtyLegalFieldNameCheck(String fieldName) {
+ return ! fieldName.contains(".");
+ }
+
@Override
public void process(boolean validate, boolean documentsOnly) {
SDDocumentType document = search.getDocument();
@@ -32,8 +37,11 @@ public class AddExtraFieldsToDocument extends Processor {
for (Field field : search.extraFieldList()) {
addSdField(search, document, (SDField)field, validate);
}
+ //TODO Vespa 8 or sooner we should avoid the dirty addition of fields from dirty 'default' summary to document at all
for (SummaryField field : search.getSummary("default").getSummaryFields()) {
- addSummaryField(search, document, field, validate);
+ if (dirtyLegalFieldNameCheck(field.getName())) {
+ addSummaryField(search, document, field, validate);
+ }
}
}
}