source providers always force a sequential reader to load the fields

jimczi · jimczi · commit b69e817a210f · 2025-05-22T13:21:42.000+01:00
diff --git a/server/src/main/java/org/elasticsearch/search/lookup/ConcurrentSegmentSourceProvider.java b/server/src/main/java/org/elasticsearch/search/lookup/ConcurrentSegmentSourceProvider.java
@@ -32,7 +32,8 @@ class ConcurrentSegmentSourceProvider implements SourceProvider {
 
     ConcurrentSegmentSourceProvider(SourceLoader loader, boolean loadSource) {
         this.sourceLoader = loader;
-        this.storedFieldLoader = StoredFieldLoader.create(loadSource, sourceLoader.requiredStoredFields());
+        // we force a sequential reader here since it is used during query execution where documents are scanned sequentially
+        this.storedFieldLoader = StoredFieldLoader.create(loadSource, sourceLoader.requiredStoredFields(), true);
     }
 
     @Override

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,8 @@ class ConcurrentSegmentSourceProvider implements SourceProvider {`
`32`	`32`
`33`	`33`	`ConcurrentSegmentSourceProvider(SourceLoader loader, boolean loadSource) {`
`34`	`34`	`this.sourceLoader = loader;`
`35`		`- this.storedFieldLoader = StoredFieldLoader.create(loadSource, sourceLoader.requiredStoredFields());`
	`35`	`+ // we force a sequential reader here since it is used during query execution where documents are scanned sequentially`
	`36`	`+ this.storedFieldLoader = StoredFieldLoader.create(loadSource, sourceLoader.requiredStoredFields(), true);`
`36`	`37`	`}`
`37`	`38`
`38`	`39`	`@Override`