re-order constructor

martijnvg · martijnvg · commit c675a94661a0 · 2025-09-01T12:06:10.000+02: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
@@ -38,8 +38,8 @@ class ConcurrentSegmentSourceProvider implements SourceProvider {
     ConcurrentSegmentSourceProvider(MappingLookup lookup, SourceFilter filter, SourceFieldMetrics metrics) {
         this.sourceLoaderProvider = sourceFilter -> lookup.newSourceLoader(sourceFilter, metrics);
         this.sourceLoader = sourceLoaderProvider.apply(filter);
-        this.isStoredSource = lookup.isSourceSynthetic() == false;
         // we force a sequential reader here since it is used during query execution where documents are scanned sequentially
+        this.isStoredSource = lookup.isSourceSynthetic() == false;
         this.storedFieldLoader = StoredFieldLoader.create(isStoredSource, sourceLoader.requiredStoredFields(), true);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -38,8 +38,8 @@ class ConcurrentSegmentSourceProvider implements SourceProvider {`
`38`	`38`	`ConcurrentSegmentSourceProvider(MappingLookup lookup, SourceFilter filter, SourceFieldMetrics metrics) {`
`39`	`39`	`this.sourceLoaderProvider = sourceFilter -> lookup.newSourceLoader(sourceFilter, metrics);`
`40`	`40`	`this.sourceLoader = sourceLoaderProvider.apply(filter);`
`41`		`- this.isStoredSource = lookup.isSourceSynthetic() == false;`
`42`	`41`	`// we force a sequential reader here since it is used during query execution where documents are scanned sequentially`
	`42`	`+ this.isStoredSource = lookup.isSourceSynthetic() == false;`
`43`	`43`	`this.storedFieldLoader = StoredFieldLoader.create(isStoredSource, sourceLoader.requiredStoredFields(), true);`
`44`	`44`	`}`
`45`	`45`