From 7b3c03043d15d9e8f9e7a4763b5161e7cf0b8ead Mon Sep 17 00:00:00 2001 From: Tor Egge Date: Fri, 3 Nov 2023 16:07:04 +0100 Subject: Adjust comment: add rationale for writing to summary field source. --- .../src/main/java/com/yahoo/schema/processing/IndexingOutputs.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'config-model/src/main/java/com/yahoo/schema/processing') diff --git a/config-model/src/main/java/com/yahoo/schema/processing/IndexingOutputs.java b/config-model/src/main/java/com/yahoo/schema/processing/IndexingOutputs.java index 385e030130b..071c2878ae8 100644 --- a/config-model/src/main/java/com/yahoo/schema/processing/IndexingOutputs.java +++ b/config-model/src/main/java/com/yahoo/schema/processing/IndexingOutputs.java @@ -137,7 +137,12 @@ public class IndexingOutputs extends Processor { for (String fieldName : summaryFields) { ret.add(new SummaryExpression(fieldName)); } - // Write to summary field source + /* + * Write to summary field source. AddExtraFieldsToDocument processor adds the "copy" + * summary transform to summary fields without a corresponding explicitly declared + * document field (2023-11-01). Future vespa versions will stop adding document + * fields for those summary fields. + */ if (!summaryFields.contains(field.getName())) { ret.add(new SummaryExpression(field.getName())); } -- cgit v1.2.3