Skip to content

ESQL: remove incorrect inline stats pruning (#141056)#142389

Merged
elasticsearchmachine merged 4 commits intoelastic:9.3from
astefan:141056_9.3_backport
Feb 13, 2026
Merged

ESQL: remove incorrect inline stats pruning (#141056)#142389
elasticsearchmachine merged 4 commits intoelastic:9.3from
astefan:141056_9.3_backport

Conversation

@astefan
Copy link
Contributor

@astefan astefan commented Feb 12, 2026

Backports #141056

(cherry picked from commit 244538b)

There are some small differences from the original PR and they are due to these two PRs that are 9.4 only:
#137923
#140238

Removes incorrect pruning in `inline stats` that lacked taking into
consideration the fact that groupings can be multi-valued. Fixes
elastic#139359 (comment)
(grouping by an mv fields can lead to inconsistencies in certain
queries) Fixes elastic#139359
Fixes elastic#140757

This also extracts a bunch of tests methods from the huge
`LogicalPlanOptimizerTests` and moves them to a new test class -
`PruneColumnsTests` which should include most of the tests that have in
the spotlight the `PruneColumns` optimization rule.

(cherry picked from commit 244538b)
@astefan astefan added >bug backport auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) >tech debt :Analytics/ES|QL AKA ESQL labels Feb 12, 2026
@elasticsearchmachine elasticsearchmachine merged commit 40e0879 into elastic:9.3 Feb 13, 2026
35 checks passed
@astefan astefan deleted the 141056_9.3_backport branch February 13, 2026 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug >tech-debt v9.3.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants