-
Notifications
You must be signed in to change notification settings - Fork 488
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
[DOC] Fix OpenSearch Java documentation for IndexSettings #4072
Comments
|
@Naarcha-AWS I would like to work on this issue |
@harshavamsi, @tobycaulk-safely, I am currently reviewing this issue and checking source code for version 2.11 and the classes you mentioned that do not exist indeed exist in the base code. Here some links: Class Class Which OpenSearch version did you use to get the BUG you mentioned? Regards, Samuel |
@harshavamsi, @tobycaulk-safely, Sorry for bothering again. I would appreciate your input on this issue. Just to see how to proceed. Regards, Samuel |
@BigSamu Sorry, my team abandoned this project a while ago so I'm not sure of the exact version. Those search results you're seeing are from a forked version of this repository, though (seems to be finding results in this repository: https://github.com/imRishN/opensearch-java). Did you search all of Github or just OpenSearch's repository? This search indicates that Whereas the (Also, sorry about the duplicate notifications. I just realized I had initially replied from my personal Github account. Oops!) |
Many thanks for your reply. You're right. My bad. I was looking at the fork of imRishN. I already looked at the source code of OpenSearch, and you and @harshavamsi were right. PR #5354 was just created to solve this error in the documentation. @Naarcha-AWS would appreciate your review of this new PR for this issue. Regards, Samuel |
What do you want to do?
Tell us about your request. Provide a summary of the request and all versions that are affected.
https://opensearch.org/docs/latest/clients/java/#creating-an-index -- this references a class
IndexSettingsBody
that does not exist in the Java client. This should just beIndexSettings
.What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
Coming from opensearch-project/opensearch-java#475
The text was updated successfully, but these errors were encountered: