Skip to content

Conversation

@alex-spies
Copy link
Contributor

Manual backport of #121193

@alex-spies alex-spies added >enhancement release highlight backport auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) test-release Trigger CI checks against release build :Analytics/ES|QL AKA ESQL v9.0.0 labels Jan 30, 2025
This effectively releases LOOKUP JOIN into tech preview. Docs will follow in a separate PR.

* Enable the lexing/grammar for LOOKUP JOIN in non-snapshot builds.
* Remove the grammar for the unsupported | JOIN ... command (without LOOKUP as first keyword). The way the lexer modes work, otherwise we'd also have to enable | JOIN ... syntax on non-snapshot builds and would have to add additional validation to provide appropriate error messages.
* Remove grammar for LOOKUP JOIN index AS ... because qualifiers are not yet supported. Otherwise we'd have to put in additional validation as well to prevent such queries.

---------

Co-authored-by: Bogdan Pintea <[email protected]>
Co-authored-by: elasticsearchmachine <[email protected]>
Co-authored-by: Nik Everett <[email protected]>
@nik9000
Copy link
Member

nik9000 commented Jan 30, 2025

check part 3 is the same failure we were ok ignoring in the original PR: #121193 (comment) - so we're ok ignoring this PR as well.

@nik9000
Copy link
Member

nik9000 commented Jan 30, 2025

The release tests passed but didn't report back. The part 3 release test is a known failure. This one is good to merge. Well, as good as we're going to get today.

@nik9000 nik9000 merged commit ea98e9d into elastic:9.0 Jan 30, 2025
15 of 18 checks passed
@alex-spies alex-spies deleted the backport-lu-join-9.0 branch January 30, 2025 17:48
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 >feature test-release Trigger CI checks against release build v9.0.1 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants