Avoid unnecessary `TaskExecutionTimeTrackingEsThreadPoolExecutor` usage

elastic / elasticsearch

Free and Open, Distributed, RESTful Search Engine

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

Other

68.59k stars 24.36k forks source link

Avoid unnecessary `TaskExecutionTimeTrackingEsThreadPoolExecutor` usage #98248

Open DaveCTurner opened 11 months ago

DaveCTurner commented 11 months ago

Today we declare several threadpools with nontrivial TaskTrackingConfig, but only use the task-tracking in certain specialized autoscaling setups. I wonder whether we should take more care to avoid this extra overhead unless we're going to use the tracking.

elasticsearchmachine commented 11 months ago

Pinging @elastic/es-distributed (Team:Distributed)

DaveCTurner commented 11 months ago

We discussed this briefly today and agreed that in due course we'll either remove this overhead or we'll start to use the tracking more widely.