-
Notifications
You must be signed in to change notification settings - Fork 198
Updated opensearch-py to reflect the latest OpenSearch API spec #864
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
base: main
Are you sure you want to change the base?
Conversation
6be850b
to
65f5f18
Compare
65f5f18
to
2afa704
Compare
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.
Backwards compatibility broken here.
@@ -2451,6 +2451,7 @@ async def search( | |||
) | |||
async def search_shards( | |||
self, | |||
body: Any = None, |
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.
This needs to be reordered after index for backwards compat.
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.
Do you mean after the last arg (i.e. after headers
) to make it backward compatible with apps using positional args for this function?
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.
Not sure what the old signature was without looking, but this change is not backwards compatible.
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.
The latest commits don' t seem to incorporate the body
argument to search_shards
, see also #898.
9f85595
to
e345f66
Compare
59cdb9a
to
4f2ccca
Compare
8072b3f
to
098b030
Compare
b8e6196
to
126edae
Compare
8259464
to
66d987d
Compare
7e11851
to
128b9ad
Compare
b1c4c10
to
634effa
Compare
Is this till being worked on? It contains the missing |
700613c
to
7cf4bfb
Compare
cb51d88
to
cd4b1be
Compare
337eacd
to
1a45136
Compare
@SimonCW AFAIK no, the spec has a lot of changes, so someone needs to fix tests that are failing here before it can be merged, add OpenSearch 3.0, etc. |
24a68a3
to
a9e043a
Compare
b4f14d1
to
e546ed3
Compare
…-05-21) Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
e546ed3
to
5de337b
Compare
Updated opensearch-py to reflect the latest OpenSearch API spec
Date: 2025-05-22