Remove unnecessary copy of filterWrappers

ChrisHegarty · ChrisHegarty · commit 15651f4839bc · 2025-01-03T10:12:12.000Z
diff --git a/server/src/main/java/org/elasticsearch/search/aggregations/bucket/filter/FiltersAggregator.java b/server/src/main/java/org/elasticsearch/search/aggregations/bucket/filter/FiltersAggregator.java
@@ -469,7 +469,7 @@ private final class MultiFilterCompetitiveLeafCollector extends AbstractLeafColl
         ) {
             super(sub, numFilters, totalNumKeys, true, hasOtherBucket);
             assert filterWrappers.isEmpty() == false;
-            disjunctionDisi = DisjunctionDISIApproximation.of(List.copyOf(filterWrappers), Long.MAX_VALUE);
+            disjunctionDisi = DisjunctionDISIApproximation.of(filterWrappers, Long.MAX_VALUE);
         }
 
         public void collect(int doc, long bucket) throws IOException {

Original file line number	Diff line number	Diff line change
`@@ -469,7 +469,7 @@ private final class MultiFilterCompetitiveLeafCollector extends AbstractLeafColl`
`469`	`469`	`) {`
`470`	`470`	`super(sub, numFilters, totalNumKeys, true, hasOtherBucket);`
`471`	`471`	`assert filterWrappers.isEmpty() == false;`
`472`		`- disjunctionDisi = DisjunctionDISIApproximation.of(List.copyOf(filterWrappers), Long.MAX_VALUE);`
	`472`	`+ disjunctionDisi = DisjunctionDISIApproximation.of(filterWrappers, Long.MAX_VALUE);`
`473`	`473`	`}`
`474`	`474`
`475`	`475`	`public void collect(int doc, long bucket) throws IOException {`