Use correct block stored fields reader

jordan-powers · jordan-powers · commit c0ec26ad4924 · 2025-06-12T20:48:44.000-07:00
diff --git a/modules/mapper-extras/src/main/java/org/elasticsearch/index/mapper/extras/MatchOnlyTextFieldMapper.java b/modules/mapper-extras/src/main/java/org/elasticsearch/index/mapper/extras/MatchOnlyTextFieldMapper.java
@@ -364,7 +364,7 @@ public Query phrasePrefixQuery(TokenStream stream, int slop, int maxExpansions,
         @Override
         public BlockLoader blockLoader(BlockLoaderContext blContext) {
             if (textFieldType.isSyntheticSource()) {
-                return new BlockStoredFieldsReader.BytesFromStringsBlockLoader(storedFieldNameForSyntheticSource());
+                return new BlockStoredFieldsReader.BytesFromBytesRefsBlockLoader(storedFieldNameForSyntheticSource());
             }
             SourceValueFetcher fetcher = SourceValueFetcher.toString(blContext.sourcePaths(name()));
             // MatchOnlyText never has norms, so we have to use the field names field

Original file line number	Diff line number	Diff line change
`@@ -364,7 +364,7 @@ public Query phrasePrefixQuery(TokenStream stream, int slop, int maxExpansions,`
`364`	`364`	`@Override`
`365`	`365`	`public BlockLoader blockLoader(BlockLoaderContext blContext) {`
`366`	`366`	`if (textFieldType.isSyntheticSource()) {`
`367`		`- return new BlockStoredFieldsReader.BytesFromStringsBlockLoader(storedFieldNameForSyntheticSource());`
	`367`	`+ return new BlockStoredFieldsReader.BytesFromBytesRefsBlockLoader(storedFieldNameForSyntheticSource());`
`368`	`368`	`}`
`369`	`369`	`SourceValueFetcher fetcher = SourceValueFetcher.toString(blContext.sourcePaths(name()));`
`370`	`370`	`// MatchOnlyText never has norms, so we have to use the field names field`