Skip to content

Fixes esql class cast bug in STATS at runtime level #138085

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fixes esql class cast bug in STATS at runtime level #138085

wants to merge 1 commit into from
+8 −20

Conversation

@ncordon
Copy link
Contributor

@ncordon ncordon commented Nov 14, 2025

Addresses the runtime side of #133992 and #136598. Builds on top of #137511

Queries like:

from airports 
rename scalerank AS x 
stats  a = count(x), b = count(x) + count(x), c = count_distinct(x)

They should never fail at runtime even if the plan was not optimal for repeated aggregations.

@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.3.0 labels Nov 14, 2025
@ncordon ncordon changed the title Fixes esql class cast bug in STATS at planning level Fixes esql class cast bug in STATS at runtime level Nov 14, 2025
@kingherc kingherc added the :Search Relevance/ES|QL Search functionality in ES|QL label Nov 17, 2025
@elasticsearchmachine elasticsearchmachine added Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch and removed needs:triage Requires assignment of a team area label labels Nov 17, 2025
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Nov 17, 2025

Pinging @elastic/es-search-relevance (Team:Search Relevance)

@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Nov 17, 2025

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Nov 17, 2025
@ncordon ncordon force-pushed the stats-fix-classCastException-runtime branch from 9d9cfdb to 1d96c16 Compare November 18, 2025 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

:Analytics/ES|QL AKA ESQL :Search Relevance/ES|QL Search functionality in ES|QL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v9.3.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@ncordon @elasticsearchmachine @kingherc @jilldoty-elastic @dnhatn