diff options
author | Jon Bratseth <bratseth@gmail.com> | 2021-10-21 16:33:33 +0200 |
---|---|---|
committer | Jon Bratseth <bratseth@gmail.com> | 2021-10-21 16:33:33 +0200 |
commit | 2f68b1b2ce21f0fb1dfe68dc2f09aa0113b7a0d8 (patch) | |
tree | 225961948f5fa0988772bdd8debcdd067246bb4f /config-model/src/main/java/com/yahoo/searchdefinition/derived | |
parent | c46b73e2271d133d096784b0ff41ac27011f70fc (diff) |
Use map to allow shadowing parent fields
Diffstat (limited to 'config-model/src/main/java/com/yahoo/searchdefinition/derived')
3 files changed, 5 insertions, 5 deletions
diff --git a/config-model/src/main/java/com/yahoo/searchdefinition/derived/SummaryClass.java b/config-model/src/main/java/com/yahoo/searchdefinition/derived/SummaryClass.java index 5f88ddba5f5..68966d39d7d 100644 --- a/config-model/src/main/java/com/yahoo/searchdefinition/derived/SummaryClass.java +++ b/config-model/src/main/java/com/yahoo/searchdefinition/derived/SummaryClass.java @@ -63,7 +63,7 @@ public class SummaryClass extends Derived { } private void deriveFields(Schema schema, DocumentSummary summary) { - for (SummaryField summaryField : summary.getSummaryFields()) { + for (SummaryField summaryField : summary.getSummaryFields().values()) { if (!accessingDiskSummary && schema.isAccessingDiskSummary(summaryField)) { accessingDiskSummary = true; } diff --git a/config-model/src/main/java/com/yahoo/searchdefinition/derived/SummaryMap.java b/config-model/src/main/java/com/yahoo/searchdefinition/derived/SummaryMap.java index cf182a1afbc..c65ed7dc762 100644 --- a/config-model/src/main/java/com/yahoo/searchdefinition/derived/SummaryMap.java +++ b/config-model/src/main/java/com/yahoo/searchdefinition/derived/SummaryMap.java @@ -39,7 +39,7 @@ public class SummaryMap extends Derived implements SummarymapConfig.Producer { } private void derive(DocumentSummary documentSummary) { - for (SummaryField summaryField : documentSummary.getSummaryFields()) { + for (SummaryField summaryField : documentSummary.getSummaryFields().values()) { if (summaryField.getTransform()== SummaryTransform.NONE) continue; if (summaryField.getTransform()==SummaryTransform.ATTRIBUTE || diff --git a/config-model/src/main/java/com/yahoo/searchdefinition/derived/VsmSummary.java b/config-model/src/main/java/com/yahoo/searchdefinition/derived/VsmSummary.java index a2c90bc92f5..4ce486e13ba 100644 --- a/config-model/src/main/java/com/yahoo/searchdefinition/derived/VsmSummary.java +++ b/config-model/src/main/java/com/yahoo/searchdefinition/derived/VsmSummary.java @@ -18,7 +18,7 @@ import java.util.*; */ public class VsmSummary extends Derived implements VsmsummaryConfig.Producer { - private Map<SummaryField, List<String>> summaryMap = new java.util.LinkedHashMap<>(1); + private final Map<SummaryField, List<String>> summaryMap = new java.util.LinkedHashMap<>(1); public VsmSummary(Schema schema) { derive(schema); @@ -31,8 +31,8 @@ public class VsmSummary extends Derived implements VsmsummaryConfig.Producer { } private void derive(Schema schema, DocumentSummary documentSummary) { - if (documentSummary==null) return; - for (SummaryField summaryField : documentSummary.getSummaryFields()) { + if (documentSummary == null) return; + for (SummaryField summaryField : documentSummary.getSummaryFields().values()) { List<String> from = toStringList(summaryField.sourceIterator()); if (doMapField(schema, summaryField)) { |