diff options
author | Tor Egge <Tor.Egge@online.no> | 2023-01-27 12:16:08 +0100 |
---|---|---|
committer | Tor Egge <Tor.Egge@online.no> | 2023-01-27 12:16:08 +0100 |
commit | 395366495d0ad2d10060aac9d76e78e2401400aa (patch) | |
tree | 5b179cce26cedee2dec37a088d745fc49f1717d9 /config-model | |
parent | 1aeb347bb3ac1c1f01c452e00ef6055cd0ead057 (diff) |
Update comment for DynamicSummaryTransformUtils.getSource.
Diffstat (limited to 'config-model')
-rw-r--r-- | config-model/src/main/java/com/yahoo/schema/processing/DynamicSummaryTransformUtils.java | 6 |
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(); |