-
Notifications
You must be signed in to change notification settings - Fork 177
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
Introduced service time metrics to OpenSearch-Py client. #716
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #716 +/- ##
==========================================
- Coverage 71.95% 71.91% -0.05%
==========================================
Files 91 94 +3
Lines 8001 8085 +84
==========================================
+ Hits 5757 5814 +57
- Misses 2244 2271 +27 ☔ View full report in Codecov by Sentry. |
Signed-off-by: saimedhi <[email protected]>
715828f
to
5247b67
Compare
The failing tests are not related to this PR but are caused by recent changes to the OpenSearch API Specification, which is now natively written in OpenAPI. The generator and developer guide will be fixed accordingly in an upcoming PR. |
@VachaShah please take a look |
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.
Rebase? This LGTM.
Signed-off-by: Sai Medhini Reddy Maryada <[email protected]>
Signed-off-by: saimedhi <[email protected]>
Signed-off-by: saimedhi <[email protected]>
Ready for review |
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.
Type Metrics
when passed around, metrics: Metrics
.
Signed-off-by: saimedhi <[email protected]>
859e385
to
a625715
Compare
…project#716) * Introduced service time metrics to opensearch-py client Signed-off-by: saimedhi <[email protected]> * Introduced service time metrics to opensearch-py client Signed-off-by: saimedhi <[email protected]> * Introduced service time metrics to opensearch-py client Signed-off-by: saimedhi <[email protected]> * Added service time metrics Signed-off-by: saimedhi <[email protected]> --------- Signed-off-by: saimedhi <[email protected]> Signed-off-by: Sai Medhini Reddy Maryada <[email protected]> Signed-off-by: AbitraryYu <[email protected]>
…project#716) * Introduced service time metrics to opensearch-py client Signed-off-by: saimedhi <[email protected]> * Introduced service time metrics to opensearch-py client Signed-off-by: saimedhi <[email protected]> * Introduced service time metrics to opensearch-py client Signed-off-by: saimedhi <[email protected]> * Added service time metrics Signed-off-by: saimedhi <[email protected]> --------- Signed-off-by: saimedhi <[email protected]> Signed-off-by: Sai Medhini Reddy Maryada <[email protected]>
Description
Introduced service time metrics to OpenSearch-Py client.
Issues Resolved
#678
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.