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

Prioritize FetchPhase over QueryPhase Under High Search Workload #39891

Closed
atris opened this issue Mar 11, 2019 · 2 comments
Closed

Prioritize FetchPhase over QueryPhase Under High Search Workload #39891

atris opened this issue Mar 11, 2019 · 2 comments
Labels
:Search/Search Search-related issues that do not fall into other categories Team:Search Meta label for search team

Comments

@atris
Copy link

atris commented Mar 11, 2019

Currently, FetchPhase and QueryPhase will be scheduled on the same threadpool. This can cause issues when dealing with search heavy workloads since FetchPhase can stall behind a slew of QueryPhases. That is problematic since FetchPhase is heavier than QueryPhase (more open search handles for eg), and also means that we have completed the QueryPhase for the specific query, hence the major heavylifting for that query has already been done.

On coordinator nodes, we should complete FetchPhase faster to reduce the probability of throttling kicking in and killing the FetchPhase, hence wasting the resources expended for completing the QueryPhase for this query (#38586). On data nodes, completing FetchPhase earlier should reduce query latencies and should also reduce pressure on memory (#29366).

Another mechanism is to schedule QueryPhase and FetchPhase on different threadpools, but that is discouraged due to the fact that we already have a large number of threadpools today.

@jimczi jimczi added the :Search/Search Search-related issues that do not fall into other categories label Mar 11, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

@javanna
Copy link
Member

javanna commented Nov 16, 2022

Closing as duplicate of #29366 .

@javanna javanna closed this as not planned Won't fix, can't repro, duplicate, stale Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Search/Search Search-related issues that do not fall into other categories Team:Search Meta label for search team
Projects
None yet
Development

No branches or pull requests

5 participants