Skip to content

Avoid subqueries in from command in snapshot mode#144334

Merged
GalLalouche merged 2 commits intoelastic:mainfrom
burqen:ap/2026.03.16.fix-AnalyzerUnmappedGoldenTests
Mar 17, 2026
Merged

Avoid subqueries in from command in snapshot mode#144334
GalLalouche merged 2 commits intoelastic:mainfrom
burqen:ap/2026.03.16.fix-AnalyzerUnmappedGoldenTests

Conversation

@burqen
Copy link
Copy Markdown
Contributor

@burqen burqen commented Mar 16, 2026

These tests assume support for subqueries in from commands, which is only enabled in snapshot builds.

Before, tests would fail with

ParsingException: line 1:48: no viable alternative at input '('

These tests assume support for subqueries in from commands, which is
only enabled in snapshot builds.

Before, tests would fail with
```
ParsingException: line 1:48: no viable alternative at input '('
```
@burqen burqen requested a review from GalLalouche March 16, 2026 15:59
@burqen burqen added >test Issues or PRs that are addressing/adding tests :Analytics/ES|QL AKA ESQL v9.4.0 labels Mar 16, 2026
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Mar 16, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

@burqen
Copy link
Copy Markdown
Contributor Author

burqen commented Mar 16, 2026

Hi @GalLalouche , would you mind having a look at these changes. They are made in collaboration with Cursor. I have zero experience so please don't trust anything ;)

Copy link
Copy Markdown
Contributor

@GalLalouche GalLalouche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @burqen!

@GalLalouche GalLalouche merged commit 4042877 into elastic:main Mar 17, 2026
37 checks passed
@GalLalouche
Copy link
Copy Markdown
Contributor

I went ahead and merged this so it wouldn't block us.

elasticsearchmachine pushed a commit that referenced this pull request Mar 17, 2026
michalborek pushed a commit to michalborek/elasticsearch that referenced this pull request Mar 23, 2026
These tests assume support for subqueries in from commands, which is
only enabled in snapshot builds.

Before, tests would fail with
```
ParsingException: line 1:48: no viable alternative at input '('
```
michalborek pushed a commit to michalborek/elasticsearch that referenced this pull request Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants