ESQL: Catch parsing exception by costin · Pull Request #124958 · elastic/elasticsearch

costin · 2025-03-16T01:22:54Z

When an invalid popMode occurs (due to an invalid query), ANTLR throws
an out-of-channel exception, bypassing the existing checks.
This PR extends the checks and properly reports the error back to the
user

Fix #119025

elasticsearchmachine · 2025-03-16T01:23:18Z

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

elasticsearchmachine · 2025-03-16T01:23:18Z

Pinging @elastic/kibana-esql (ES|QL-ui)

elasticsearchmachine · 2025-03-16T01:23:19Z

Hi @costin, I've created a changelog YAML for you.

When an invalid popMode occurs (due to an invalid query), ANTLR throws an out-of-channel exception, bypassing the existing checks. This PR extends the checks and properly reports the error back to the user Fix elastic#119025

fang-xing-esql

LGTM!

elasticsearchmachine · 2025-03-16T03:38:46Z

💔 Backport failed

Status	Branch	Result
❌	8.18	Commit could not be cherrypicked due to conflicts
❌	8.x	Commit could not be cherrypicked due to conflicts
❌	9.0	Commit could not be cherrypicked due to conflicts
❌	8.17	Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 124958

When an invalid popMode occurs (due to an invalid query), ANTLR throws an out-of-channel exception, bypassing the existing checks. This PR extends the checks and properly reports the error back to the user Fix elastic#119025 (cherry picked from commit dc15462) # Conflicts: # x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/EsqlParser.java

When an invalid popMode occurs (due to an invalid query), ANTLR throws an out-of-channel exception, bypassing the existing checks. This PR extends the checks and properly reports the error back to the user Fix elastic#119025 (cherry picked from commit dc15462) # Conflicts: # x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/EsqlParser.java # x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/parser/StatementParserTests.java

costin · 2025-03-16T20:46:01Z

💚 All backports created successfully

Status	Branch	Result
✅	8.x
✅	9.0
✅	8.18
✅	8.17

Questions ?

Please refer to the Backport tool documentation

When an invalid popMode occurs (due to an invalid query), ANTLR throws an out-of-channel exception, bypassing the existing checks. This PR extends the checks and properly reports the error back to the user Fix elastic#119025 (cherry picked from commit dc15462) # Conflicts: # x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/EsqlParser.java # x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/parser/StatementParserTests.java

costin · 2025-03-16T21:08:43Z

💚 All backports created successfully

Status	Branch	Result
✅	9.0

Questions ?

Please refer to the Backport tool documentation

When an invalid popMode occurs (due to an invalid query), ANTLR throws an out-of-channel exception, bypassing the existing checks. This PR extends the checks and properly reports the error back to the user Fix elastic#119025 (cherry picked from commit dc15462) # Conflicts: # x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/EsqlParser.java # x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/parser/StatementParserTests.java

costin · 2025-03-17T00:38:12Z

💚 All backports created successfully

Status	Branch	Result
✅	9.0

Questions ?

Please refer to the Backport tool documentation

When an invalid popMode occurs (due to an invalid query), ANTLR throws an out-of-channel exception, bypassing the existing checks. This PR extends the checks and properly reports the error back to the user Fix elastic#119025 (cherry picked from commit dc15462) # Conflicts: # x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/EsqlParser.java # x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/parser/StatementParserTests.java

costin · 2025-03-17T00:51:12Z

💚 All backports created successfully

Status	Branch	Result
✅	8.18
✅	8.17

Questions ?

Please refer to the Backport tool documentation

When an invalid popMode occurs (due to an invalid query), ANTLR throws an out-of-channel exception, bypassing the existing checks. This PR extends the checks and properly reports the error back to the user Fix elastic#119025 (cherry picked from commit dc15462) # Conflicts: # x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/EsqlParser.java # x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/parser/StatementParserTests.java

When an invalid popMode occurs (due to an invalid query), ANTLR throws an out-of-channel exception, bypassing the existing checks. This PR extends the checks and properly reports the error back to the user Fix elastic#119025 (cherry picked from commit dc15462) # Conflicts: # x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/EsqlParser.java

costin · 2025-03-17T00:59:13Z

💚 All backports created successfully

Status	Branch	Result
✅	8.x

Questions ?

Please refer to the Backport tool documentation

* ESQL: Catch parsing exception (#124958) When an invalid popMode occurs (due to an invalid query), ANTLR throws an out-of-channel exception, bypassing the existing checks. This PR extends the checks and properly reports the error back to the user Fix #119025 (cherry picked from commit dc15462) # Conflicts: # x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/EsqlParser.java * Update StatementParserTests.java Fix test for 8.x branch

* ESQL: Catch parsing exception (#124958) When an invalid popMode occurs (due to an invalid query), ANTLR throws an out-of-channel exception, bypassing the existing checks. This PR extends the checks and properly reports the error back to the user Fix #119025 (cherry picked from commit dc15462) # Conflicts: # x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/EsqlParser.java # x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/parser/StatementParserTests.java * Update StatementParserTests.java Fix test for 8.x branch

idegtiarenko · 2025-03-17T08:21:33Z

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/EsqlParser.java

            throw new ParsingException("ESQL statement is too large, causing stack overflow when generating the parsing tree: [{}]", query);
+            // likely thrown by an invalid popMode (such as extra closing parenthesis)
+        } catch (EmptyStackException ese) {
+            throw new ParsingException("Invalid query [{}]", query);


Unfortunately exception is very generic and does not bring any details on position,
however I wonder if we could make any assumptions as for what went wrong here?

It might be frustrating to see message like this for not too complex query like:

stats min = min(a), max = max(a)) WHERE (a % 3 > 10 OR a / 2 > 100), avg = avg(a) WHERE a / 2 > 100 BY a

note max(a)).

We know this could be caused by extra ) or ]. I wonder if we should attempt to count them in here to highlight this could be an issue?

* main: (95 commits) Mute org.elasticsearch.datastreams.lifecycle.DataStreamLifecycleServiceIT testLifecycleAppliedToFailureStore elastic#124999 Merge template mappings properly during validation (elastic#124784) [Build] Rework internal build plugin plugin to work with Isolated Projects (elastic#123461) [Build] Require reason for usesDefaultDistribution (elastic#124707) Mute org.elasticsearch.packaging.test.DockerTests test011SecurityEnabledStatus elastic#124990 Mute org.elasticsearch.xpack.ilm.TimeSeriesDataStreamsIT testRolloverAction elastic#124987 Mute org.elasticsearch.packaging.test.BootstrapCheckTests test10Install elastic#124957 Mute org.elasticsearch.integration.DataStreamLifecycleServiceRuntimeSecurityIT testRolloverLifecycleAndForceMergeAuthorized elastic#124978 Mute org.elasticsearch.xpack.esql.action.CrossClusterAsyncQueryStopIT testStopQuery elastic#124977 Mute org.elasticsearch.xpack.esql.action.CrossClusterAsyncQueryStopIT testStopQueryLocal elastic#121672 Mention zero-window state in networking docs (elastic#124967) Remove remoteAddress field from TransportResponse (elastic#120016) Include failures in partial response (elastic#124929) Prevent work starvation bug if using scaling EsThreadPoolExecutor with core pool size = 0 (elastic#124732) Re-enable analysis stemmer test (elastic#124961) Mute org.elasticsearch.xpack.esql.action.CrossClusterAsyncQueryStopIT testStopQueryLocalNoRemotes elastic#124959 ESQL: Catch parsing exception (elastic#124958) ESQL: Improve error message for ( and [ (elastic#124177) Mute org.elasticsearch.xpack.esql.qa.single_node.EsqlSpecIT test {lookup-join.MvJoinKeyFromRow SYNC} elastic#124951 Mute org.elasticsearch.datastreams.lifecycle.DataStreamLifecycleServiceIT testErrorRecordingOnRetention elastic#124950 ... # Conflicts: # server/src/main/java/org/elasticsearch/index/mapper/vectors/DenseVectorFieldMapper.java # server/src/test/java/org/elasticsearch/index/mapper/vectors/DenseVectorFieldTypeTests.java

When an invalid popMode occurs (due to an invalid query), ANTLR throws an out-of-channel exception, bypassing the existing checks. This PR extends the checks and properly reports the error back to the user Fix elastic#119025

costin added >bug auto-backport Automatically create backport pull requests when merged :Analytics/ES|QL AKA ESQL ES|QL-ui Impacts ES|QL UI v8.18.1 v8.19.0 v9.0.1 v9.1.0 v8.17.4 labels Mar 16, 2025

costin requested review from bpintea and fang-xing-esql March 16, 2025 01:22

elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Mar 16, 2025

costin added 2 commits March 15, 2025 18:37

ESQL: Catch parsing exception

a07a514

When an invalid popMode occurs (due to an invalid query), ANTLR throws an out-of-channel exception, bypassing the existing checks. This PR extends the checks and properly reports the error back to the user Fix elastic#119025

Add changelog

3327ebf

costin force-pushed the fix/119025 branch from a7a71c1 to 3327ebf Compare March 16, 2025 01:39

[CI] Auto commit changes from spotless

b26bfaa

fang-xing-esql approved these changes Mar 16, 2025

View reviewed changes

costin merged commit dc15462 into elastic:main Mar 16, 2025
17 checks passed

costin deleted the fix/119025 branch March 16, 2025 03:37

elasticsearchmachine added the backport pending label Mar 16, 2025

costin mentioned this pull request Mar 16, 2025

[8.x] ESQL: Catch parsing exception (#124958) #124972

Closed

costin mentioned this pull request Mar 16, 2025

[9.0] ESQL: Catch parsing exception (#124958) #124973

Closed

costin mentioned this pull request Mar 16, 2025

[8.18] ESQL: Catch parsing exception (#124958) #124974

Closed

costin mentioned this pull request Mar 16, 2025

[8.17] ESQL: Catch parsing exception (#124958) #124975

Closed

costin mentioned this pull request Mar 16, 2025

[9.0] ESQL: Catch parsing exception (#124958) #124976

Closed

costin mentioned this pull request Mar 17, 2025

[9.0] ESQL: Catch parsing exception (#124958) #124980

Closed

costin mentioned this pull request Mar 17, 2025

[8.18] ESQL: Catch parsing exception (#124958) #124981

Merged

costin mentioned this pull request Mar 17, 2025

[8.17] ESQL: Catch parsing exception (#124958) #124982

Merged

costin mentioned this pull request Mar 17, 2025

[8.x] ESQL: Catch parsing exception (#124958) #124983

Merged

idegtiarenko reviewed Mar 17, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ESQL: Catch parsing exception#124958

ESQL: Catch parsing exception#124958
costin merged 3 commits intoelastic:mainfrom
costin:fix/119025

costin commented Mar 16, 2025

Uh oh!

elasticsearchmachine commented Mar 16, 2025

Uh oh!

elasticsearchmachine commented Mar 16, 2025

Uh oh!

elasticsearchmachine commented Mar 16, 2025

Uh oh!

fang-xing-esql left a comment

Uh oh!

Uh oh!

elasticsearchmachine commented Mar 16, 2025

Uh oh!

costin commented Mar 16, 2025

Uh oh!

costin commented Mar 16, 2025

Uh oh!

costin commented Mar 17, 2025

Uh oh!

costin commented Mar 17, 2025

Uh oh!

costin commented Mar 17, 2025

Uh oh!

idegtiarenko Mar 17, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

costin commented Mar 16, 2025

Uh oh!

elasticsearchmachine commented Mar 16, 2025

Uh oh!

elasticsearchmachine commented Mar 16, 2025

Uh oh!

elasticsearchmachine commented Mar 16, 2025

Uh oh!

fang-xing-esql left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

elasticsearchmachine commented Mar 16, 2025

💔 Backport failed

Uh oh!

costin commented Mar 16, 2025

💚 All backports created successfully

Questions ?

Uh oh!

costin commented Mar 16, 2025

💚 All backports created successfully

Questions ?

Uh oh!

costin commented Mar 17, 2025

💚 All backports created successfully

Questions ?

Uh oh!

costin commented Mar 17, 2025

💚 All backports created successfully

Questions ?

Uh oh!

costin commented Mar 17, 2025

💚 All backports created successfully

Questions ?

Uh oh!

idegtiarenko Mar 17, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants