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

Delete warmer api to support acknowledgements #3833

Closed
javanna opened this issue Oct 4, 2013 · 0 comments
Closed

Delete warmer api to support acknowledgements #3833

javanna opened this issue Oct 4, 2013 · 0 comments

Comments

@javanna
Copy link
Member

javanna commented Oct 4, 2013

The delete warmer api always returns "acknowledged":true, as it doesn't support any acknowledgement mechanism when it comes to updating the cluster state. It currently returns after the master has applied the change locally, which doesn't necessarily mean that all the other nodes already know about the new cluster state version.

We then need to add support for acknowledgements from the other nodes and the timeout parameter, which refers to the maximum wait for acknowledgements (default 10s as in the other apis) in a backward compatible manner.

@ghost ghost assigned javanna Oct 4, 2013
javanna added a commit to javanna/elasticsearch that referenced this issue Oct 15, 2013
Added support for acknowledgements in delete warmer api using the generic mechanism introduced in#3786

Closes elastic#3833
javanna added a commit that referenced this issue Oct 15, 2013
Added support for acknowledgements in delete warmer api using the generic mechanism introduced in#3786

Closes #3833
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
Added support for acknowledgements in delete warmer api using the generic mechanism introduced in#3786

Closes elastic#3833
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant