Skip to content

Conversation

@nik9000
Copy link
Member

@nik9000 nik9000 commented Oct 29, 2025

The bug still exists in the underlying code, but it isn't released so I'm working around it by doing an eager call to hashCode. We'll fix the real bug real soon.

Closes #137338

The bug still exists in the underlying code, but it isn't released so
I'm working around it by doing an eager call to `hashCode`. We'll fix
the real bug real soon.

Closes elastic#137338
@nik9000 nik9000 requested a review from JonasKunz October 29, 2025 19:50
@nik9000 nik9000 added >test Issues or PRs that are addressing/adding tests :Analytics/ES|QL AKA ESQL v9.3.0 auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Oct 29, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Oct 29, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

rawValues
);
/*
* hashcode mutates the histogram by doing some lazy work.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here I mean it makes something lazily. I don't mean that we were lazy when we wrote it. It's a lot of code....

@elasticsearchmachine elasticsearchmachine merged commit 15a1472 into elastic:main Oct 29, 2025
34 checks passed
@nik9000 nik9000 deleted the 137338 branch October 29, 2025 20:58
shmuelhanoch pushed a commit to shmuelhanoch/elasticsearch that referenced this pull request Oct 29, 2025
The bug still exists in the underlying code, but it isn't released so
I'm working around it by doing an eager call to `hashCode`. We'll fix
the real bug real soon.

Closes elastic#137338
chrisparrinello pushed a commit to chrisparrinello/elasticsearch that referenced this pull request Nov 3, 2025
The bug still exists in the underlying code, but it isn't released so
I'm working around it by doing an eager call to `hashCode`. We'll fix
the real bug real soon.

Closes elastic#137338
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!) Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] ShowExecSerializationTests testConcurrentSerialization failing

2 participants