Skip to content

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

@opensearch-trigger-bot opensearch-trigger-bot bot commented Dec 2, 2024

Updated opensearch-py to reflect the latest OpenSearch API spec
Date: 2025-05-22

Copy link
Member

@dblock dblock left a 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,
Copy link
Member

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.

Copy link
Contributor

@nhtruong nhtruong Mar 26, 2025

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?

Copy link
Member

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.

Copy link

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.

@opensearch-trigger-bot opensearch-trigger-bot bot force-pushed the automated-api-update branch 6 times, most recently from 9f85595 to e345f66 Compare December 19, 2024 03:41
@opensearch-trigger-bot opensearch-trigger-bot bot force-pushed the automated-api-update branch 2 times, most recently from 59cdb9a to 4f2ccca Compare December 22, 2024 03:38
@opensearch-trigger-bot opensearch-trigger-bot bot force-pushed the automated-api-update branch 5 times, most recently from 8072b3f to 098b030 Compare January 3, 2025 03:38
@opensearch-trigger-bot opensearch-trigger-bot bot force-pushed the automated-api-update branch 3 times, most recently from b8e6196 to 126edae Compare January 6, 2025 03:39
@opensearch-trigger-bot opensearch-trigger-bot bot force-pushed the automated-api-update branch 3 times, most recently from 7e11851 to 128b9ad Compare April 3, 2025 03:42
@opensearch-trigger-bot opensearch-trigger-bot bot force-pushed the automated-api-update branch 3 times, most recently from b1c4c10 to 634effa Compare April 12, 2025 03:41
@SimonCW
Copy link

SimonCW commented Apr 14, 2025

Is this till being worked on? It contains the missing plugins.ml APIs.

@opensearch-trigger-bot opensearch-trigger-bot bot force-pushed the automated-api-update branch 3 times, most recently from 700613c to 7cf4bfb Compare April 17, 2025 03:42
@opensearch-trigger-bot opensearch-trigger-bot bot force-pushed the automated-api-update branch 3 times, most recently from cb51d88 to cd4b1be Compare April 29, 2025 03:43
@opensearch-trigger-bot opensearch-trigger-bot bot force-pushed the automated-api-update branch 5 times, most recently from 337eacd to 1a45136 Compare May 8, 2025 03:46
@dblock
Copy link
Member

dblock commented May 8, 2025

@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.

@opensearch-trigger-bot opensearch-trigger-bot bot force-pushed the automated-api-update branch 3 times, most recently from 24a68a3 to a9e043a Compare May 13, 2025 03:46
@opensearch-trigger-bot opensearch-trigger-bot bot force-pushed the automated-api-update branch 3 times, most recently from b4f14d1 to e546ed3 Compare May 20, 2025 03:46
…-05-21)

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants