-
Notifications
You must be signed in to change notification settings - Fork 25k
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
Stored script related docs of Java API are outdated #28144
Comments
@danielmitterdorfer That is correct. I'll will update these docs. |
@martijnvg the docs are still showing outdated content for release version 5.1 |
@ankishagarwal If I recall correctly for 5.x, only the docs for 5.6 have been updated. Most of the times if doc changes are backported to 5.x, the changes are only backported to 5.6, since that is the only 5.x version that is maintained. |
The docs of release 5.6 is similar to 5.1 i.e outdated. It asks user to use classes PutIndexedScriptResponse, GetIndexedScriptResponse , and DeleteIndexedScriptResponse rather than asking for PutStoredScriptResponse ,etc. |
I was mistaken this doc change was only backported to 6.0, so I backported it to 5.6 branch. This should be visible soon. |
Originally reported in #28114:
PutIndexedScriptResponse, GetIndexedScriptResponse, and DeleteIndexedScriptResponse are outdated in https://www.elastic.co/guide/en/elasticsearch/client/java-api/master/indexed-scripts.html
PutIndexedScriptResponse = client.preparePutIndexedScript()...
GetIndexedScriptResponse = client.prepareGetIndexedScript()...
DeleteIndexedScriptResponse = client.prepareDeleteIndexedScript()...
These are leftovers from #17650 and all
*IndexedScript*
should be replaced by*StoredScript*
if I understood the change correctly (e.g.PutIndexedScriptResponse
->PutStoredScriptResponse
,client.preparePutIndexedScript()
->client.preparePutStoredScript()
).@martijnvg I hope it's ok I'm assigning it to you as you did the change and might remember best.
The text was updated successfully, but these errors were encountered: