Integrate TDigestState with circuit breakers

elastic / elasticsearch

Free and Open, Distributed, RESTful Search Engine

https://www.elastic.co/products/elasticsearch

Other

68.71k stars 24.4k forks source link

Integrate TDigestState with circuit breakers #99815

Open kkrik-es opened 10 months ago

kkrik-es commented 10 months ago

Description

TDigestis missing integration with circuit breakers. The latter can track memory allocation and throw an exception before exhausting the heap. While the latest (forked) version of the TDigest library has constant memory footprint of < 100kB, it's still possible to create thousands of such instances (one per bucket) so it's still valuable to complete this task.

elasticsearchmachine commented 10 months ago

Pinging @elastic/es-analytics-geo (Team:Analytics)

elasticsearchmachine commented 1 month ago

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