-
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
Search latency tracking - Coordinator node #8386
Merged
msfroh
merged 61 commits into
opensearch-project:main
from
buddharajusahil:CoordinatorStats
Sep 20, 2023
Merged
Search latency tracking - Coordinator node #8386
msfroh
merged 61 commits into
opensearch-project:main
from
buddharajusahil:CoordinatorStats
Sep 20, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
buddharajusahil
requested review from
reta,
anasalkouz,
andrross,
Bukhtawar,
CEHENKLE,
dblock,
gbbafna,
setiah,
kartg,
kotwanikunal,
mch2,
nknize,
owaiskazi19,
Rishikesh1159,
ryanbogan,
saratvemulapalli,
shwetathareja,
dreamer-89,
tlfeng,
VachaShah,
dbwiddis and
sachinpkale
as code owners
June 30, 2023 21:40
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:
|
buddharajusahil
changed the title
Coordinator stats
Search latency tracking - Coordinator node
Jul 5, 2023
Gradle Check (Jenkins) Run Completed with:
|
buddharajusahil
force-pushed
the
CoordinatorStats
branch
from
July 6, 2023 21:23
2118dff
to
b54c492
Compare
Signed-off-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com>
Removing unused variable from SearchRequestStats
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Signed-off-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com>
Fixing broken UT
Signed-off-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com>
Fixing javadoc issue
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Signed-off-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com>
Removing RequestStats file as not needed
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
msfroh
approved these changes
Sep 20, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @sgup432! Looks good to me.
I'm going to keep retrying the Gradle check to see if we can get a build without the flaky test failures.
Gradle Check (Jenkins) Run Completed with:
|
sarthakaggarwal97
pushed a commit
to sarthakaggarwal97/OpenSearch
that referenced
this pull request
Sep 24, 2023
We are trying to add search stats to coordinator level node stats. This keeps track of the total time, current requests, and total requests of each request phase. Also added support for general coordinator stats as well on the node level. Signed-off-by: sahil buddharaju <sahilbud@amazon.com> Signed-off-by: sahil <61558528+buddharajusahil@users.noreply.github.com> Signed-off-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com> Signed-off-by: Sagar <99425694+sgup432@users.noreply.github.com> Co-authored-by: sahil buddharaju <sahilbud@amazon.com> Co-authored-by: Sagar Upadhyaya <upasagar@amazon.com> Co-authored-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com> Co-authored-by: Sagar <99425694+sgup432@users.noreply.github.com>
brusic
pushed a commit
to brusic/OpenSearch
that referenced
this pull request
Sep 25, 2023
We are trying to add search stats to coordinator level node stats. This keeps track of the total time, current requests, and total requests of each request phase. Also added support for general coordinator stats as well on the node level. Signed-off-by: sahil buddharaju <sahilbud@amazon.com> Signed-off-by: sahil <61558528+buddharajusahil@users.noreply.github.com> Signed-off-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com> Signed-off-by: Sagar <99425694+sgup432@users.noreply.github.com> Co-authored-by: sahil buddharaju <sahilbud@amazon.com> Co-authored-by: Sagar Upadhyaya <upasagar@amazon.com> Co-authored-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com> Co-authored-by: Sagar <99425694+sgup432@users.noreply.github.com> Signed-off-by: Ivan Brusic <ivan.brusic@flocksafety.com>
sgup432
added a commit
to sgup432/OpenSearch
that referenced
this pull request
Sep 29, 2023
We are trying to add search stats to coordinator level node stats. This keeps track of the total time, current requests, and total requests of each request phase. Also added support for general coordinator stats as well on the node level. Signed-off-by: sahil buddharaju <sahilbud@amazon.com> Signed-off-by: sahil <61558528+buddharajusahil@users.noreply.github.com> Signed-off-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com> Signed-off-by: Sagar <99425694+sgup432@users.noreply.github.com> Co-authored-by: sahil buddharaju <sahilbud@amazon.com> Co-authored-by: Sagar Upadhyaya <upasagar@amazon.com> Co-authored-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com> Co-authored-by: Sagar <99425694+sgup432@users.noreply.github.com> Signed-off-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com>
This was referenced Sep 29, 2023
msfroh
pushed a commit
that referenced
this pull request
Sep 29, 2023
vikasvb90
pushed a commit
to vikasvb90/OpenSearch
that referenced
this pull request
Oct 10, 2023
We are trying to add search stats to coordinator level node stats. This keeps track of the total time, current requests, and total requests of each request phase. Also added support for general coordinator stats as well on the node level. Signed-off-by: sahil buddharaju <sahilbud@amazon.com> Signed-off-by: sahil <61558528+buddharajusahil@users.noreply.github.com> Signed-off-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com> Signed-off-by: Sagar <99425694+sgup432@users.noreply.github.com> Co-authored-by: sahil buddharaju <sahilbud@amazon.com> Co-authored-by: Sagar Upadhyaya <upasagar@amazon.com> Co-authored-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com> Co-authored-by: Sagar <99425694+sgup432@users.noreply.github.com>
Closed
4 tasks
1 task
shiv0408
pushed a commit
to Gaurav614/OpenSearch
that referenced
this pull request
Apr 25, 2024
We are trying to add search stats to coordinator level node stats. This keeps track of the total time, current requests, and total requests of each request phase. Also added support for general coordinator stats as well on the node level. Signed-off-by: sahil buddharaju <sahilbud@amazon.com> Signed-off-by: sahil <61558528+buddharajusahil@users.noreply.github.com> Signed-off-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com> Signed-off-by: Sagar <99425694+sgup432@users.noreply.github.com> Co-authored-by: sahil buddharaju <sahilbud@amazon.com> Co-authored-by: Sagar Upadhyaya <upasagar@amazon.com> Co-authored-by: Sagar Upadhyaya <sagar.upadhyaya.121@gmail.com> Co-authored-by: Sagar <99425694+sgup432@users.noreply.github.com> Signed-off-by: Shivansh Arora <hishiv@amazon.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
This PR is addressing this issue: #8381
We are trying to add search stats to coordinator level node stats. This keeps track of the total time, current requests, and total requests of each request phase.
Also added support for general coordinator stats as well on the node level.
Manual Testing:
Case 1:
First command:
1 shard test
Insert Data:
Search Query
This is an unsuccessful search request
We will grab the node stats now with this command:
curl -XGET 'localhost:9200/_nodes/runTask-0/stats/indices/search?pretty=true'
This is the expected output, as the phases after query will take minimal time due to the query phase resulting in 0 hits
If we instead run this query:
We get this response for the coordinator stats:
So now if we run both queries subsequently, we get these coordinator stats:
This makes sense because each individual query took 34 and 7 ms respectively, and it has correctly added up these stats.
Case 2:
In this case, we will perform a DFS query then fetch action instead of query then fetch action. We will populate the date in the same way with the same settings in case 1 except with 5 shards.
The command we will use is:
The response we get when we ask for coordinator stats is now:
This is as expected. If we then run a normal query then fetch from above:
Now the coordinator stats are:
Performance Testing:
Latency:
Throughput Comparison:
Operation Counts:
CPU Usage:
Garbage Collection: