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] ML Commons Java Rest Client #2788

Open
saratvemulapalli opened this issue Jul 30, 2024 · 2 comments
Open

[FEATURE] ML Commons Java Rest Client #2788

saratvemulapalli opened this issue Jul 30, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@saratvemulapalli
Copy link
Member

saratvemulapalli commented Jul 30, 2024

Is your feature request related to a problem?
I have a custom distribution of OpenSearch where I have ML Commons installed in its own dedicated cluter.
I would like to have ML Commons Java Rest Client in order to have my application code to communicate with ML Commons for predict.

What solution would you like?
As said above I would like a ML Commons rest client which can be consumed by my application and I would be happy to contribute.

I see 4 options:

  1. In addition to transport ml-client[1], another new interface will be introduced for rest client and a default implementation will be contributed.
  2. The default implementation would be a wrapper around opensearch-high-level-rest client (which is on its path to deprecation).
  3. The default implementation would be a wrapper around Apache HTTP client.
  4. OpenSearch-java which is a replacement for opensearch-high-level-rest client. Though it does not support plugins yet, contributing to https://github.com/opensearch-project/opensearch-api-specification/blob/main/spec/namespaces/ml.yaml will automatically generate a Java client (this is work in progress by @Xtansia / @dblock)

Looking at possible options, I do prefer 4., maintaining one spec and generating clients in the language of choice. In my use case, I would need opensearch-java client.

Maintainers, let me know your thoughts/feedback?

@saratvemulapalli saratvemulapalli added enhancement New feature or request untriaged labels Jul 30, 2024
@dblock dblock removed the untriaged label Aug 19, 2024
@dblock
Copy link
Member

dblock commented Aug 19, 2024

Catch All Triage - 1, 2, 3

@dhrubo-os
Copy link
Collaborator

@saratvemulapalli if we follow option 4, do we need have any code development for rest client or the spec should have that support already?

I assigned the issue to you for now;

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: On-deck
Development

No branches or pull requests

3 participants