Skip to content

Conversation

nik9000
Copy link
Member

@nik9000 nik9000 commented Feb 23, 2025

They are so so so slow that this is effectively a bad bug.

They are so so so slow that this is effectively a bad bug.
@nik9000 nik9000 added >docs General docs changes :Analytics/ES|QL AKA ESQL v8.16.4 labels Feb 23, 2025
Copy link
Contributor

Documentation preview:

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@elasticsearchmachine elasticsearchmachine added v8.16.5 Team:Docs Meta label for docs team Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Feb 23, 2025
@elasticsearchmachine
Copy link
Collaborator

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


ES|QL::

* `VALUES` aggregate function can run for a long, long time when collecting many, many groups. This can consume a thread for days at the worst case. Fixed by {es-pull}#123073[#123073] and available in 8.16.5, 8.17.3, 8.18.0, and all releases after that.
Copy link
Contributor

Choose a reason for hiding this comment

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

@nik9000 any way we can quantify "many" here?

Copy link
Member Author

Choose a reason for hiding this comment

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

It's n^2 so it's a sliding scale. But millions of buckets can run for days. Hundreds of thousands of buckets can run for many minutes. I'll push something with that in it.

@nik9000 nik9000 requested a review from leemthompo February 24, 2025 14:36
@nik9000
Copy link
Member Author

nik9000 commented Feb 24, 2025

@leemthompo, could you have another look?

@nik9000 nik9000 enabled auto-merge (squash) February 24, 2025 16:16
@nik9000
Copy link
Member Author

nik9000 commented Feb 24, 2025

I've applied the copy editing and set it to auto-merge. I'll drag it into 8.17 as well and auto-merge-without-review that one.

@nik9000
Copy link
Member Author

nik9000 commented Feb 24, 2025

8.17: #123288

@nik9000 nik9000 merged commit e12d777 into elastic:8.16 Feb 24, 2025
4 of 5 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 >docs General docs changes Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) Team:Docs Meta label for docs team v8.16.4 v8.16.5 v8.17.3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants