Skip to content

Commit dd29fec

Browse files
committed
Fix test
1 parent 939705c commit dd29fec

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/optimizer/LogicalPlanOptimizerTests.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8013,19 +8013,17 @@ public void testMultipleKnnQueriesInPrefilters() {
80138013

80148014
// Second OR (keyword == "test" OR knn2)
80158015
var secondOr = as(and.right(), Or.class);
8016-
var keywordEq = as(secondOr.left(), Equals.class);
8016+
as(secondOr.left(), Equals.class);
80178017
var secondKnn = as(secondOr.right(), Knn.class);
80188018

80198019
// First KNN should have the second OR as its filter
80208020
List<Expression> firstKnnFilters = firstKnn.filterExpressions();
80218021
assertThat(firstKnnFilters.size(), equalTo(1));
8022-
var secondOrWithoutKnn = secondOr.replaceChildren(List.of(secondOr.left(), Literal.TRUE));
8023-
assertTrue(firstKnnFilters.contains(secondOrWithoutKnn));
8022+
assertTrue(firstKnnFilters.contains(secondOr.left()));
80248023

80258024
// Second KNN should have the first OR as its filter
80268025
List<Expression> secondKnnFilters = secondKnn.filterExpressions();
80278026
assertThat(secondKnnFilters.size(), equalTo(1));
8028-
var firstOrWithoutFilters = firstOr.replaceChildren(List.of(Literal.TRUE, firstOr.right()));
8029-
assertTrue(secondKnnFilters.contains(firstOrWithoutFilters));
8027+
assertTrue(secondKnnFilters.contains(firstOr.right()));
80308028
}
80318029
}

0 commit comments

Comments
 (0)