diff options
author | Tor Egge <Tor.Egge@yahooinc.com> | 2022-07-14 12:07:20 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-07-14 12:07:20 +0200 |
commit | 555aea2e364d730ca1d6970e348c919bb7e1c16a (patch) | |
tree | d0db9131d1f28cec29d66fbd6f893f1cbd03064e | |
parent | a648dbbe7c188f32fb6089d1bd54be248e034b97 (diff) | |
parent | 687ca05627398682fa7e2cd5b7ec87dddc88361a (diff) |
Merge pull request #23502 from vespa-engine/geirst/remove-textextractor-summary-transform
Remove textextractor summary transform.
3 files changed, 1 insertions, 3 deletions
diff --git a/config-model/src/main/java/com/yahoo/schema/derived/SummaryMap.java b/config-model/src/main/java/com/yahoo/schema/derived/SummaryMap.java index 55dc252f273..9d3d00f1481 100644 --- a/config-model/src/main/java/com/yahoo/schema/derived/SummaryMap.java +++ b/config-model/src/main/java/com/yahoo/schema/derived/SummaryMap.java @@ -69,7 +69,7 @@ public class SummaryMap extends Derived implements SummarymapConfig.Producer { // This works, but is suboptimal. We could consolidate to a minimal set and // use the right value from the minimal set as the third parameter here, // and add "override" commands to multiple static values - boolean useFieldNameAsArgument = summaryField.getTransform().isDynamic() || summaryField.getTransform() == SummaryTransform.TEXTEXTRACTOR; + boolean useFieldNameAsArgument = summaryField.getTransform().isDynamic(); resultTransforms.put(summaryField.getName(), new FieldResultTransform(summaryField.getName(), summaryField.getTransform(), useFieldNameAsArgument ? summaryField.getName() : "")); diff --git a/config-model/src/main/java/com/yahoo/schema/processing/AddExtraFieldsToDocument.java b/config-model/src/main/java/com/yahoo/schema/processing/AddExtraFieldsToDocument.java index 471d338a5f6..3afc25131c0 100644 --- a/config-model/src/main/java/com/yahoo/schema/processing/AddExtraFieldsToDocument.java +++ b/config-model/src/main/java/com/yahoo/schema/processing/AddExtraFieldsToDocument.java @@ -38,7 +38,6 @@ public class AddExtraFieldsToDocument extends Processor { case BOLDED: case DYNAMICBOLDED: case DYNAMICTEASER: - case TEXTEXTRACTOR: case DOCUMENT_ID: // TODO: Adding the 'documentid' field should no longer be needed when the docsum framework in the backend has been simplified and the transform is always used. addSummaryField(schema, document, summaryField, validate); break; diff --git a/config-model/src/main/java/com/yahoo/vespa/documentmodel/SummaryTransform.java b/config-model/src/main/java/com/yahoo/vespa/documentmodel/SummaryTransform.java index 5c3aef89edc..e94518c988d 100644 --- a/config-model/src/main/java/com/yahoo/vespa/documentmodel/SummaryTransform.java +++ b/config-model/src/main/java/com/yahoo/vespa/documentmodel/SummaryTransform.java @@ -18,7 +18,6 @@ public enum SummaryTransform { POSITIONS("positions"), RANKFEATURES("rankfeatures"), SUMMARYFEATURES("summaryfeatures"), - TEXTEXTRACTOR("textextractor"), GEOPOS("geopos"), ATTRIBUTECOMBINER("attributecombiner"), MATCHED_ELEMENTS_FILTER("matchedelementsfilter"), |