Skip to content

[8.x] ESQL: Skip lookup fields when eliminating missing fields (#118658)#118836

Merged
elasticsearchmachine merged 1 commit intoelastic:8.xfrom
alex-spies:backport/8.x/pr-118658
Dec 17, 2024
Merged

[8.x] ESQL: Skip lookup fields when eliminating missing fields (#118658)#118836
elasticsearchmachine merged 1 commit intoelastic:8.xfrom
alex-spies:backport/8.x/pr-118658

Conversation

@alex-spies
Copy link
Contributor

Backports the following commits to 8.x:

)

We do not have SearchStats for fields from lookup indices. And unfortunately, these are hard to obtain.

For now, just do not apply ReplaceMissingFieldWithNull to fields coming from an index used in LOOKUP JOIN. These are identified via their indexmode.
@alex-spies alex-spies added :Analytics/ES|QL AKA ESQL >non-issue 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) labels Dec 17, 2024
@elasticsearchmachine elasticsearchmachine merged commit 6d5e436 into elastic:8.x Dec 17, 2024
@alex-spies alex-spies deleted the backport/8.x/pr-118658 branch December 17, 2024 13:18
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 >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants