[9.3] ESQL: Fix injected attributes's IDs in UnionAll branches (#141262)#141675
Merged
elasticsearchmachine merged 3 commits intoelastic:9.3from Feb 2, 2026
Merged
[9.3] ESQL: Fix injected attributes's IDs in UnionAll branches (#141262)#141675elasticsearchmachine merged 3 commits intoelastic:9.3from
elasticsearchmachine merged 3 commits intoelastic:9.3from
Conversation
) This fixes the generation of name IDs for the attributes corresponding to the unmapped fields and are pushed to different branches in `UntionAll`. So far, one set of IDs was generated and reused for all subplans. This is now updated to individual set per subplan. Along the change, the handling of `Fork` in `ResolveUnmapped` has been somewhat simplified. Also, more unit tests have been completed (where the plans are simple enough) and the plan comments updated to replace the `EsqlProject` with the now merged `Project`. A minor collateral proposed change: the CSV spec-based tests skipped due to missing capabilities are now logged. (cherry picked from commit 8e3113c)
9.3 does not have elastic#139058, so the implicit limits at the top of subquery branches are still in place. Adjust the expectations accordingly.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This will backport the following commits from
mainto9.3: