-
Notifications
You must be signed in to change notification settings - Fork 185
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
Integrated generated dangling_indices client APIs into the existing module #511
Integrated generated dangling_indices client APIs into the existing module #511
Conversation
0dec59d
to
3512ee1
Compare
Codecov Report
@@ Coverage Diff @@
## main #511 +/- ##
=======================================
Coverage 70.87% 70.87%
=======================================
Files 83 83
Lines 7790 7790
=======================================
Hits 5521 5521
Misses 2269 2269
|
the dangling index. | ||
:arg cluster_manager_timeout: Operation timeout for connection | ||
to cluster-manager node. | ||
:arg master_timeout: Operation timeout for connection to master |
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.
Similar comment for master_timeout comments to mention that it is deprecated. I think it applies to the other generated code PRs.
…odule, ensuring alignment with the server and maintaining backward compatibility Signed-off-by: saimedhi <[email protected]>
3512ee1
to
febb5c0
Compare
Signed-off-by: Sai Medhini Reddy Maryada <[email protected]>
@@ -8,6 +8,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) | |||
### Changed | |||
- Integrated generated `tasks client` APIs into the existing module, ensuring alignment with the server and maintaining backward compatibility ([#508](https://github.com/opensearch-project/opensearch-py/pull/508)) | |||
- Integrated generated `ingest client` APIs into the existing module, ensuring alignment with the server and maintaining backward compatibility ([#513](https://github.com/opensearch-project/opensearch-py/pull/513)) | |||
- Integrated generated `dangling_indices client` APIs into the existing module, ensuring alignment with the server and maintaining backward compatibility ([#511](https://github.com/opensearch-project/opensearch-py/pull/511)) |
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.
As a nit, I find these changelogs a bit strange. I think what we're trying to say is that now these parts are geberated code? So maybe "Generate dangling_indices
client from API specs"?
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.
I think we also need to bring in some generated test code that exercises these APIs, even if it's to check that they can be imported without errors.
…odule, ensuring alignment with the server and maintaining backward compatibility (opensearch-project#511) Signed-off-by: saimedhi <[email protected]> Signed-off-by: Sai Medhini Reddy Maryada <[email protected]> Signed-off-by: roma2023 <[email protected]>
Description
Integrated generated dangling_indices client APIs into the existing module, ensuring alignment with the server and maintaining backward compatibility
Issues Resolved
Related to #477
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.