-
Notifications
You must be signed in to change notification settings - Fork 732
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
[5.2] Index::setSettings broken? #1295
Comments
@nickygerritsen Thanks for reporting. I think you are right. Could you open a PR with a bug fix? @ewgRa FYI |
@nickygerritsen I think you are right. Can you make pull request? It is trivial, and would be nice to cover this case by test. |
On it! |
ruflin
pushed a commit
that referenced
this issue
May 5, 2017
mhernik
pushed a commit
to mhernik/Elastica
that referenced
this issue
Jul 24, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It seems that in 5.2+
setSettings
on anIndex
is broken. It tries to set the settings on the whole cluster instead of only the index.In 5.1 this still works.
If I take a short look at the code it seems
Elasticsearch\Endpoints\Cluster\Settings\Put
is the wrong class and we should useElasticsearch\Endpoints\Indices\Settings\Put
instead?The text was updated successfully, but these errors were encountered: