Skip to content

[9.3] Deep copy BytesRef when creating a constant vector block (#141242)#141530

Merged
elasticsearchmachine merged 1 commit intoelastic:9.3from
dnhatn:backport/9.3/pr-141242
Jan 29, 2026
Merged

[9.3] Deep copy BytesRef when creating a constant vector block (#141242)#141530
elasticsearchmachine merged 1 commit intoelastic:9.3from
dnhatn:backport/9.3/pr-141242

Conversation

@dnhatn
Copy link
Copy Markdown
Member

@dnhatn dnhatn commented Jan 29, 2026

Backports the following commits to 9.3:

)

Currently, when creating a constant BytesRef block or vector from a 
constant input, we do not deep copy the BytesRef in the evaluator. As a
result, if the evaluator generates additional blocks afterwards, these
blocks may share the underlying bytes with the constant block, leading
to incorrect data. This issue was seen in tests where two rows had the
same data when they should not.

Closes elastic#140809
Closes elastic#140621
Closes elastic#140615
@dnhatn dnhatn 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 Jan 29, 2026
@elasticsearchmachine elasticsearchmachine merged commit 148ddbc into elastic:9.3 Jan 29, 2026
35 checks passed
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.3.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants