Skip to content

[8.19] ESQL: Fix wildcard DROP after LOOKUP JOIN (#130448) #130696

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

alex-spies
Copy link
Contributor

@alex-spies alex-spies commented Jul 7, 2025

This will backport the following commits from main to 8.19:

In EsqlSession#fieldNames, keep track of wildcard patterns in DROP
separately from the references and wildcard patterns in KEEP commands.

Only the latter are relevant to determine if we need to ask for all fields for the
lookup index in the field caps request.

(cherry picked from commit 05fc6f2)

# Conflicts:
#	x-pack/plugin/esql/qa/testFixtures/src/main/resources/lookup-join.csv-spec
#	x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/action/EsqlCapabilities.java
#	x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/session/IndexResolverFieldNamesTests.java
@alex-spies alex-spies added >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Analytics/ES|QL AKA ESQL labels Jul 7, 2025
@elasticsearchmachine elasticsearchmachine merged commit 6288fe2 into elastic:8.19 Jul 7, 2025
22 checks passed
@alex-spies alex-spies deleted the backport/8.19/pr-130448 branch July 7, 2025 10:05
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 >bug v8.19.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants