Skip to content

ES|QL "ROW a=null | SORT a" crashes with AssertionError #121535

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
Closed
jan-elastic opened this issue Feb 3, 2025 · 1 comment · Fixed by #122762
Closed

ES|QL "ROW a=null | SORT a" crashes with AssertionError #121535

jan-elastic opened this issue Feb 3, 2025 · 1 comment · Fixed by #122762
Labels
:Analytics/ES|QL AKA ESQL >bug Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)

Comments

@jan-elastic
Copy link
Contributor

jan-elastic commented Feb 3, 2025 •
edited
Loading

Elasticsearch Version

9.0.0

Installed Plugins

No response

Java Version

n/a

OS Version

n/a

Problem Description

Executing the ES|QL command

ROW a=null | SORT a

crashes on main with the following assertion error:

[2025-02-03T15:33:10,801][ERROR][o.e.b.ElasticsearchUncaughtExceptionHandler] [runTask-0] fatal error in thread [elasticsearch[runTask-0][search][T#1]], exiting java.lang.AssertionError: estimated row size [0] wasn't set
	at org.elasticsearch.xpack.esql.planner.LocalExecutionPlanner.planTopN(LocalExecutionPlanner.java:362)
(...)

Note: this doesn't occur in production, because assertions are disabled.

Steps to Reproduce

spin up ES, Kibana, go to discover, ES|QL mode, enter "ROW a=null | SORT a"

Logs (if relevant)

No response
@jan-elastic jan-elastic added >bug needs:triage Requires assignment of a team area label labels Feb 3, 2025
@alex-spies alex-spies added :Analytics/ES|QL AKA ESQL and removed needs:triage Requires assignment of a team area label labels Feb 3, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Feb 3, 2025
@kanoshiou kanoshiou mentioned this issue Feb 17, 2025
Merged
@dnhatn dnhatn closed this as completed in 4d2cb53 Mar 4, 2025
dnhatn pushed a commit to dnhatn/elasticsearch that referenced this issue Mar 4, 2025
@kanoshiou @dnhatn
ESQL: Ensure non-zero row size in EstimatesRowSize (elastic#122762)
8ce0d6c 
Closes elastic#121535
dnhatn pushed a commit to dnhatn/elasticsearch that referenced this issue Mar 4, 2025
@kanoshiou
ESQL: Ensure non-zero row size in EstimatesRowSize (elastic#122762)
476073f 
Closes elastic#121535

(cherry picked from commit 4d2cb53)
elasticsearchmachine pushed a commit that referenced this issue Mar 4, 2025
@dnhatn @kanoshiou
ESQL: Ensure non-zero row size in EstimatesRowSize (#122762) (#123958)
8e409d6 
Closes #121535

Co-authored-by: kanoshiou <[email protected]>
georgewallace pushed a commit to georgewallace/elasticsearch that referenced this issue Mar 11, 2025
@kanoshiou @georgewallace
ESQL: Ensure non-zero row size in EstimatesRowSize (elastic#122762)
176015c 
Closes elastic#121535
ivancea pushed a commit to ivancea/elasticsearch that referenced this issue Apr 11, 2025
@kanoshiou @dnhatn
ESQL: Ensure non-zero row size in EstimatesRowSize (elastic#122762)
5e3f34f 
Closes elastic#121535
elasticsearchmachine pushed a commit that referenced this issue Apr 11, 2025
@dnhatn @kanoshiou
@costin @ivancea
[8.x] ESQL: Ensure non-zero row size in EstimatesRowSize (#122762) (#…
ca45080 
…123959)

* ESQL: Ensure non-zero row size in `EstimatesRowSize` (#122762)

Closes #121535

(cherry picked from commit 4d2cb53)

* Removed getFirst() usages

---------

Co-authored-by: kanoshiou <[email protected]>
Co-authored-by: Costin Leau <[email protected]>
Co-authored-by: Iván Cea Fontenla <[email protected]>
Co-authored-by: Iván Cea Fontenla <[email protected]>
elasticsearchmachine pushed a commit that referenced this issue Apr 11, 2025
@dnhatn @kanoshiou
@costin @ivancea
[8.18] ESQL: Ensure non-zero row size in EstimatesRowSize (#122762) (…
47bda42 
…#123957)

* ESQL: Ensure non-zero row size in `EstimatesRowSize` (#122762)

Closes #121535

* Removed getFirest() usages

---------

Co-authored-by: kanoshiou <[email protected]>
Co-authored-by: Costin Leau <[email protected]>
Co-authored-by: Iván Cea Fontenla <[email protected]>
Co-authored-by: Iván Cea Fontenla <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Analytics/ES|QL AKA ESQL >bug Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ESQL: Ensure non-zero row size in EstimatesRowSize kanoshiou/elasticsearch
3 participants
@elasticsearchmachine @alex-spies @jan-elastic