Skip to content

Conversation

@craigtaverner
Copy link
Contributor

Manual backport of #127906

There are some analyser tests for this, but I thought it useful to also have a yaml test for more coverage, since there was temporarily an issue elastic#120189 with this, later fixed in elastic#120617.
@craigtaverner craigtaverner added >test Issues or PRs that are addressing/adding tests backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Analytics/ES|QL AKA ESQL v9.0.2 labels May 11, 2025
@elasticsearchmachine elasticsearchmachine merged commit e4e9cd0 into elastic:9.0 May 11, 2025
16 checks passed
@craigtaverner craigtaverner deleted the 9.0_esql_yaml_test_lookup_join_missing_key branch May 11, 2025 09:54
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 Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.0.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants