diff --git a/muted-tests.yml b/muted-tests.yml index 4564baffe6d1a..d6acebe6a04cd 100644 --- a/muted-tests.yml +++ b/muted-tests.yml @@ -404,18 +404,6 @@ tests: - class: org.elasticsearch.xpack.sql.qa.jdbc.single_node.SingleNodeJdbcResultSetIT method: testGettingValidLongWithoutCasting issue: https://github.com/elastic/elasticsearch/issues/145580 -- class: org.elasticsearch.xpack.esql.tree.EsqlNodeSubclassTests - method: testReplaceChildren {class org.elasticsearch.xpack.esql.plan.physical.MergeExec} - issue: https://github.com/elastic/elasticsearch/issues/145581 -- class: org.elasticsearch.xpack.esql.tree.EsqlNodeSubclassTests - method: testReplaceChildren {class org.elasticsearch.xpack.esql.plan.logical.Filter} - issue: https://github.com/elastic/elasticsearch/issues/145583 -- class: org.elasticsearch.xpack.esql.tree.EsqlNodeSubclassTests - method: testTransform {class org.elasticsearch.xpack.esql.plan.physical.ExchangeSinkExec} - issue: https://github.com/elastic/elasticsearch/issues/145584 -- class: org.elasticsearch.xpack.esql.tree.EsqlNodeSubclassTests - method: testTransform {class org.elasticsearch.xpack.esql.plan.physical.ExchangeExec} - issue: https://github.com/elastic/elasticsearch/issues/145585 - class: org.elasticsearch.xpack.esql.CsvTests method: test {csv-spec:stats_percentile.percentileOfLong} issue: https://github.com/elastic/elasticsearch/issues/145589 @@ -425,27 +413,9 @@ tests: - class: org.elasticsearch.xpack.sql.qa.jdbc.single_node.SingleNodeJdbcFetchSizeIT method: testScrollWithDatetimeAndTimezoneParam issue: https://github.com/elastic/elasticsearch/issues/145592 -- class: org.elasticsearch.xpack.esql.tree.EsqlNodeSubclassTests - method: testReplaceChildren {class org.elasticsearch.xpack.esql.plan.logical.join.InlineJoin} - issue: https://github.com/elastic/elasticsearch/issues/145594 -- class: org.elasticsearch.xpack.esql.tree.EsqlNodeSubclassTests - method: testTransform {class org.elasticsearch.xpack.esql.plan.logical.Dissect} - issue: https://github.com/elastic/elasticsearch/issues/145595 - class: org.elasticsearch.xpack.esql.plan.logical.CommandLicenseTests method: testLicenseCheck issue: https://github.com/elastic/elasticsearch/issues/145600 -- class: org.elasticsearch.xpack.esql.tree.EsqlNodeSubclassTests - method: testTransform {class org.elasticsearch.xpack.esql.plan.logical.local.ResolvingProject} - issue: https://github.com/elastic/elasticsearch/issues/145601 -- class: org.elasticsearch.xpack.esql.tree.EsqlNodeSubclassTests - method: testReplaceChildren {class org.elasticsearch.xpack.esql.plan.logical.promql.PromqlCommand} - issue: https://github.com/elastic/elasticsearch/issues/145603 -- class: org.elasticsearch.xpack.esql.tree.EsqlNodeSubclassTests - method: testReplaceChildren {class org.elasticsearch.xpack.esql.plan.logical.local.ResolvingProject} - issue: https://github.com/elastic/elasticsearch/issues/145604 -- class: org.elasticsearch.xpack.esql.tree.EsqlNodeSubclassTests - method: testTransform {class org.elasticsearch.xpack.esql.plan.physical.MergeExec} - issue: https://github.com/elastic/elasticsearch/issues/145606 - class: org.elasticsearch.indices.IndexingMemoryControllerIT method: testDeletesAloneCanTriggerRefresh issue: https://github.com/elastic/elasticsearch/issues/145609 diff --git a/x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/promql/AcrossSeriesAggregate.java b/x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/promql/AcrossSeriesAggregate.java index 47ccda5c91a11..f08f0d12ef6cd 100644 --- a/x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/promql/AcrossSeriesAggregate.java +++ b/x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/promql/AcrossSeriesAggregate.java @@ -113,8 +113,7 @@ public List output() { if (grouping == Grouping.WITHOUT) { return List.of(timeseriesAttribute); } - assert expressionsResolved() : "output() called on unresolved plan"; - return groupings.stream().filter(a -> a.dataType() != DataType.NULL).toList(); + return groupings.stream().filter(a -> a.resolved() == false || a.dataType() != DataType.NULL).toList(); } @Override