Skip to content

[9.0] Make LOOKUP JOIN docs examples fully tested (#126622) #126756

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.

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

Merged

Conversation

craigtaverner
Copy link
Contributor

Backports the following commits to 9.0:

The current LOOKUP JOIN docs include examples that are not tested by the ES|QL tests, unlike most other examples in the documentation. This PR fixes that, changing two examples to use existing tests, and adding a new csv-spec file for the remaining four examples. These four are not required to show results, so the tests have empty data and do not require any results. This means we are testing only the syntax (parsing and semantic analysis), which is sufficient for the docs.
@craigtaverner craigtaverner added :Analytics/ES|QL AKA ESQL >docs General docs changes 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) Team:Docs Meta label for docs team labels Apr 14, 2025
@elasticsearchmachine elasticsearchmachine merged commit 57ca5e1 into elastic:9.0 Apr 14, 2025
16 checks passed
@craigtaverner craigtaverner deleted the backport/9.0/pr-126622 branch April 14, 2025 08:57
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 >docs General docs changes Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) Team:Docs Meta label for docs team v9.0.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants