Skip to content

Conversation

ivancea
Copy link
Contributor

@ivancea ivancea commented Jul 21, 2025

Manual 9.1 backport of #131541

There's a TransportVersion change here

`SampleOperator.Status` wasn't declared as a NamedWritable by the plugin, leading to serialization errors when `SAMPLE` is used with `profile: true`.

It leads to an `IllegalArgumentException: Unknown NamedWriteable [org.elasticsearch.compute.operator.Operator$Status][sample]`

Profiles will be tested in this PR: elastic#131474, that's currently failing because of this bug
@ivancea ivancea added >bug backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL v9.1.1 labels Jul 21, 2025
@ivancea ivancea merged commit 9d82c48 into elastic:9.1 Jul 21, 2025
34 checks passed
@ivancea ivancea deleted the backport-9.1-fix-sample-operator-serialization branch July 21, 2025 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL backport >bug Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.1.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants