ESQL: Remove unnecessary AwaitsFix (#100543)

Update the tests and remove an AwaitsFix
elastic · Oct 9, 2023 · 22f9eaa · 22f9eaa
1 parent d2fa1ee
commit 22f9eaa
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/...src/test/java/org/elasticsearch/xpack/esql/optimizer/LocalPhysicalPlanOptimizerTests.java b/...src/test/java/org/elasticsearch/xpack/esql/optimizer/LocalPhysicalPlanOptimizerTests.java
@@ -296,7 +296,6 @@ public void testAnotherCountAllWithFilter() {
         assertThat(expected.toString(), is(esStatsQuery.query().toString()));
     }
 
-    @AwaitsFix(bugUrl = "intermediateAgg does proper reduction but the agg itself does not - the optimizer needs to improve")
     public void testMultiCountAllWithFilter() {
         var plan = plan("""
             from test
@@ -311,7 +310,7 @@ public void testMultiCountAllWithFilter() {
         var exchange = as(agg.child(), ExchangeExec.class);
         var esStatsQuery = as(exchange.child(), EsStatsQueryExec.class);
         assertThat(esStatsQuery.limit(), is(nullValue()));
-        assertThat(Expressions.names(esStatsQuery.output()), contains("count", "seen"));
+        assertThat(Expressions.names(esStatsQuery.output()), contains("count", "seen", "count", "seen", "count", "seen"));
         var expected = wrapWithSingleQuery(QueryBuilders.rangeQuery("emp_no").gt(10010), "emp_no");
         assertThat(expected.toString(), is(esStatsQuery.query().toString()));
     }