Skip to content

Conversation

bpintea
Copy link
Contributor

@bpintea bpintea commented Oct 7, 2025

Backports the following commits to 9.2:

Currently, in case the join key is (locally) null, the whole join is replaced with an `Eval` and a `Project`. The project however contain only reference attributes. This is insufficient, in case the attribute should point to another attribute pointing to a literal `null` added in the `Eval`.

This fixes that by allowing the generated `Project` to contain `NamedExpression`s, which can then be proper `Alias`es.
This is done by adjusting the join output calculation to work with the `NamedExpression`s, instead of just `Attribute`s.
@bpintea bpintea added :Analytics/ES|QL AKA ESQL >bug 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 Oct 7, 2025
@elasticsearchmachine elasticsearchmachine merged commit 0abdbd0 into elastic:9.2 Oct 7, 2025
34 checks passed
@bpintea bpintea deleted the backport/9.2/pr-135446 branch October 7, 2025 12:21
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 Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.2.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants