Skip to content
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

[FEATURE] Add ml-commons APIs in the OpenSearchClient #3574

Open
Zhangxunmt opened this issue Feb 20, 2025 · 5 comments
Open

[FEATURE] Add ml-commons APIs in the OpenSearchClient #3574

Zhangxunmt opened this issue Feb 20, 2025 · 5 comments
Assignees
Labels
enhancement New feature or request

Comments

@Zhangxunmt
Copy link
Collaborator

Is your feature request related to a problem?
When integrating ml-commons in other Java applications like data prepper, lambda, etc, it's better to use the SDSK client to interact with ml-commons. Hence, we should include ml-commons APIs in the OpenSearch Client. https://opensearch.org/docs/latest/clients/java/.

What solution would you like?
Include the ml-commons APIs in the Java Client as the default offering.

What alternatives have you considered?
We already have ml-commons in the OpenSearch-Py, but that's only for python applications.

Do you have any additional context?

@Zhangxunmt Zhangxunmt added enhancement New feature or request untriaged labels Feb 20, 2025
@ylwu-amzn
Copy link
Collaborator

@nathaliellenaa I think we can use API spec to generate Java client code. Can you confirm ?

@nathaliellenaa
Copy link
Contributor

nathaliellenaa commented Feb 21, 2025

Yes, we can use API spec to generate Java client code. This is the YAML file for generating the code from spec in opensearch-java and we can see in this PR that the ML APIs are generated in the Java client code.

@Zhangxunmt
Copy link
Collaborator Author

@nathaliellenaa sounds good! If we can make this client generated and released in the next version, the first user of this java ml-commons client will be data prepper.

@nathaliellenaa
Copy link
Contributor

Sure @Zhangxunmt, I'll make sure all ML APIs are released in the next version.

@pyek-bot
Copy link
Contributor

@Zhangxunmt let's assign this to @nathaliellenaa ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In Progress
Development

No branches or pull requests

5 participants