-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
consume query level cpu and memory usage in query insights #13739
base: main
Are you sure you want to change the base?
consume query level cpu and memory usage in query insights #13739
Conversation
❌ Gradle check result for 80c4615: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
80c4615
to
02bb39e
Compare
❌ Gradle check result for 02bb39e: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
...sights/src/main/java/org/opensearch/plugin/insights/core/listener/QueryInsightsListener.java
Outdated
Show resolved
Hide resolved
...sights/src/main/java/org/opensearch/plugin/insights/core/listener/QueryInsightsListener.java
Show resolved
Hide resolved
...va/org/opensearch/plugin/insights/rules/transport/top_queries/TransportTopQueriesAction.java
Show resolved
Hide resolved
Looks good to me overall. Some minor comments and changelog also needed. |
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.
Minor comments to improve the readability of the code!
...sights/src/main/java/org/opensearch/plugin/insights/core/listener/QueryInsightsListener.java
Outdated
Show resolved
Hide resolved
plugins/query-insights/src/main/java/org/opensearch/plugin/insights/rules/model/MetricType.java
Show resolved
Hide resolved
...sights/src/main/java/org/opensearch/plugin/insights/core/listener/QueryInsightsListener.java
Outdated
Show resolved
Hide resolved
02bb39e
to
14294b1
Compare
❌ Gradle check result for 14294b1: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
plugins/query-insights/src/main/java/org/opensearch/plugin/insights/rules/model/MetricType.java
Show resolved
Hide resolved
plugins/query-insights/src/main/java/org/opensearch/plugin/insights/rules/model/Attribute.java
Outdated
Show resolved
Hide resolved
...sights/src/main/java/org/opensearch/plugin/insights/core/listener/QueryInsightsListener.java
Outdated
Show resolved
Hide resolved
...sights/src/main/java/org/opensearch/plugin/insights/core/listener/QueryInsightsListener.java
Show resolved
Hide resolved
67074ca
to
f003f9a
Compare
❌ Gradle check result for 67074ca: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
❌ Gradle check result for f003f9a: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
...ry-insights/src/main/java/org/opensearch/plugin/insights/core/service/TopQueriesService.java
Outdated
Show resolved
Hide resolved
f003f9a
to
aa9835b
Compare
❌ Gradle check result for aa9835b: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Signed-off-by: Chenyang Ji <cyji@amazon.com>
Signed-off-by: Chenyang Ji <cyji@amazon.com>
aa9835b
to
2d19224
Compare
2d19224
to
fa2c127
Compare
❌ Gradle check result for 2d19224: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Thanks for the refactoring! LGTM |
❌ Gradle check result for fa2c127: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Signed-off-by: Chenyang Ji <cyji@amazon.com>
fa2c127
to
46669df
Compare
❌ Gradle check result for 46669df: Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
❌ Gradle check result for fa2c127: Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Description
This is a follow up on https://github.com/opensearch-project/OpenSearch/pull/13172/files#diff-916410426108bbd0b06952c34490ca05c7a2a78c1c10f7af28e88ead23a70f24 to consume query level cpu and memory usage in query insights
Related Issues
#12399
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.