Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve thread pools #18613

Open
3 of 7 tasks
jasontedor opened this issue May 27, 2016 · 2 comments
Open
3 of 7 tasks

Improve thread pools #18613

jasontedor opened this issue May 27, 2016 · 2 comments
Labels
:Core/Infra/Resiliency Keep running when everything is ok. Die quickly if things go horribly wrong. >enhancement Meta Team:Core/Infra Meta label for core/infra team

Comments

@jasontedor
Copy link
Member

jasontedor commented May 27, 2016

This issue is a meta-issue for collapsing several sub-issues for improving thread pools in core Elasticsearch:

The proposal here is that core Elasticsearch will no longer provide support for custom executors. Of course, a plugin can always create an executor on its own and so core Elasticsearch will provide an extension point for such plugins to provide stats back to core Elasticsearch for reading in node stats API and cat thread pools API.

Relates #17915, relates #18491, relates #14448, relates #15866

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@rjernst rjernst added the Team:Core/Infra Meta label for core/infra team label May 4, 2020
@rjernst rjernst added needs:triage Requires assignment of a team area label and removed needs:triage Requires assignment of a team area label labels Dec 3, 2020
@rjernst
Copy link
Member

rjernst commented Dec 9, 2020

This issue has been stalled due to complexities of the general threadpool (and others) that are currently unbounded. The concern is that bounding them may cause contention, and could (in theory?) cause deadlocks. I don't think this will be theoretically resolved. We know that having unbounded queues has problems of its own, as outlined here. I propose that we move forward now by implementing the hard bound on queues suggested here (256 * max threads) in master for 8.0. This gives us sufficient time to test through our own daily performance tests, as well as users to test in the alphas/betas for 8.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Resiliency Keep running when everything is ok. Die quickly if things go horribly wrong. >enhancement Meta Team:Core/Infra Meta label for core/infra team
Projects
None yet
Development

No branches or pull requests

3 participants