Skip to content

[9.2] ESQL: Replace any Attribute type when pushing down past Project (#135295)#136275

Merged
elasticsearchmachine merged 1 commit intoelastic:9.2from
bpintea:backport/9.2/pr-135295
Oct 9, 2025
Merged

[9.2] ESQL: Replace any Attribute type when pushing down past Project (#135295)#136275
elasticsearchmachine merged 1 commit intoelastic:9.2from
bpintea:backport/9.2/pr-135295

Conversation

@bpintea
Copy link
Copy Markdown
Contributor

@bpintea bpintea commented Oct 9, 2025

Backports the following commits to 9.2:

…tic#135295)

This will allow any attribute type be replaced in node that's being pushed down past a `Project`. So far only `ReferenceAttribute`s were replaced. Some nodes reference other types, like `FieldAttribute` (in `OrderBy`), which would otherwise be pushed down unchanged, and thus possibly wrongly (such as when the `Project` renames _to_ the field attribute name, thus showdowing it).

Closes elastic#134407
@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 9, 2025
@elasticsearchmachine elasticsearchmachine merged commit d1fd990 into elastic:9.2 Oct 9, 2025
34 checks passed
@bpintea bpintea deleted the backport/9.2/pr-135295 branch October 9, 2025 12:22
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