summaryrefslogtreecommitdiffstats
path: root/config-model
diff options
context:
space:
mode:
authorTor Egge <Tor.Egge@online.no>2023-01-27 12:16:08 +0100
committerTor Egge <Tor.Egge@online.no>2023-01-27 12:16:08 +0100
commit395366495d0ad2d10060aac9d76e78e2401400aa (patch)
tree5b179cce26cedee2dec37a088d745fc49f1717d9 /config-model
parent1aeb347bb3ac1c1f01c452e00ef6055cd0ead057 (diff)
Update comment for DynamicSummaryTransformUtils.getSource.
Diffstat (limited to 'config-model')
-rw-r--r--config-model/src/main/java/com/yahoo/schema/processing/DynamicSummaryTransformUtils.java6
1 files changed, 4 insertions, 2 deletions
diff --git a/config-model/src/main/java/com/yahoo/schema/processing/DynamicSummaryTransformUtils.java b/config-model/src/main/java/com/yahoo/schema/processing/DynamicSummaryTransformUtils.java
index 30866172f39..50739ad3883 100644
--- a/config-model/src/main/java/com/yahoo/schema/processing/DynamicSummaryTransformUtils.java
+++ b/config-model/src/main/java/com/yahoo/schema/processing/DynamicSummaryTransformUtils.java
@@ -53,8 +53,10 @@ public class DynamicSummaryTransformUtils {
}
public static String getSource(SummaryField summaryField, Schema schema) {
- // Summary fields with the original supported type is always present in the document type,
- // and we must use that field as source at run-time.
+ // Summary fields with the original supported type is always present in the document type.
+ // However, if the source of that summary field is a single explicit source that exists in the schema we
+ // use that as source instead as this is handled by the backend code.
+ // This is a move in the right direction to avoid adding some summary fields as extra document fields.
if (isOriginalSupportedType(summaryField.getDataType())) {
if (summaryField.hasExplicitSingleSource()) {
String sourceFieldName = summaryField.getSingleSource();