-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Concurrent Search Tasks Response Updates #7673
Conversation
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
c400861
to
3217749
Compare
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Failing test seems unrelated to my changes - https://build.ci.opensearch.org/job/gradle-check/16116/testReport/
|
Gradle Check (Jenkins) Run Completed with:
|
@reta Not sure why it gave me a merge conflict on that last commit but I think we should be good to squash and merge now. It's rerunning the tests but as you can see the last change has no diff. |
Gradle Check (Jenkins) Run Completed with:
|
Signed-off-by: Jay Deng <[email protected]>
Signed-off-by: Jay Deng <[email protected]>
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Looks like that latest test failure could be related to my change, diving into it. |
server/src/main/java/org/opensearch/tasks/TaskResourceStats.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Jay Deng <[email protected]>
There was an NPE issue in the constructor, should be fixed with the latest commit. |
Gradle Check (Jenkins) Run Completed with:
|
* Add average,min,max and thread info to tasks response Signed-off-by: Jay Deng <[email protected]> * Update server/src/main/java/org/opensearch/tasks/TaskResourceStats.java Signed-off-by: Jay Deng <[email protected]> --------- Signed-off-by: Jay Deng <[email protected]> (cherry picked from commit e3740f7) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Add average,min,max and thread info to tasks response Signed-off-by: Jay Deng <[email protected]> * Update server/src/main/java/org/opensearch/tasks/TaskResourceStats.java Signed-off-by: Jay Deng <[email protected]> --------- Signed-off-by: Jay Deng <[email protected]> (cherry picked from commit e3740f7) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Add average,min,max and thread info to tasks response Signed-off-by: Jay Deng <[email protected]> * Update server/src/main/java/org/opensearch/tasks/TaskResourceStats.java Signed-off-by: Jay Deng <[email protected]> --------- Signed-off-by: Jay Deng <[email protected]> (cherry picked from commit e3740f7) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Signed-off-by: Andriy Redko <[email protected]>
* Add average,min,max and thread info to tasks response Signed-off-by: Jay Deng <[email protected]> * Update server/src/main/java/org/opensearch/tasks/TaskResourceStats.java Signed-off-by: Jay Deng <[email protected]> --------- Signed-off-by: Jay Deng <[email protected]> (cherry picked from commit e3740f7) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Signed-off-by: Andriy Redko <[email protected]>
* Add average,min,max and thread info to tasks response Signed-off-by: Jay Deng <[email protected]> * Update server/src/main/java/org/opensearch/tasks/TaskResourceStats.java Signed-off-by: Jay Deng <[email protected]> --------- Signed-off-by: Jay Deng <[email protected]>
* Add average,min,max and thread info to tasks response * Update server/src/main/java/org/opensearch/tasks/TaskResourceStats.java --------- (cherry picked from commit e3740f7) Signed-off-by: Jay Deng <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Signed-off-by: Andriy Redko <[email protected]> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…pensearch-project#7856) * Add average,min,max and thread info to tasks response * Update server/src/main/java/org/opensearch/tasks/TaskResourceStats.java --------- (cherry picked from commit e3740f7) Signed-off-by: Jay Deng <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Signed-off-by: Andriy Redko <[email protected]> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Add average,min,max and thread info to tasks response Signed-off-by: Jay Deng <[email protected]> * Update server/src/main/java/org/opensearch/tasks/TaskResourceStats.java Signed-off-by: Jay Deng <[email protected]> --------- Signed-off-by: Jay Deng <[email protected]> Signed-off-by: Shivansh Arora <[email protected]>
Description
After #7502 has been merged we will start tracking the resource usage of the concurrent search threadpool. This change adds average,min,max resource usage to the tasks API response which are more meaningful in the concurrent search use case. These stats will only be shown whenever thread executions > 1.
This is an implementation of solution 1 in the design from #7425.
Related Issues
#7425
Doc update Issue: opensearch-project/documentation-website#4206
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.