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

Index Warmer Setting is not dynamic anymore on 0.90.1 #3246

Closed
jgagnon1 opened this issue Jun 26, 2013 · 0 comments
Closed

Index Warmer Setting is not dynamic anymore on 0.90.1 #3246

jgagnon1 opened this issue Jun 26, 2013 · 0 comments

Comments

@jgagnon1
Copy link

According to the documentation; Index warmup can be disabled by setting index.warmer.enabled to false. It is supported as a realtime setting using update settings API.

When I try to set it back to true;

{"index.warmer.enabled":"true"}

I get to following exception;

{"error":"RemoteTransportException[[xxx][inet[/xx.x.xx.xx:9300]][indices/settings/update]]; nested: ElasticSearchIllegalArgumentException[Can't update non dynamic settings[[index.warmer.enabled]] for open indices[[public_20120701]]]; ","status":400}
@ghost ghost assigned spinscale Jun 27, 2013
spinscale added a commit to spinscale/elasticsearch that referenced this issue Jun 27, 2013
Even though proposed in the documentation, the realtime enabling/disabling of
index warmers was not supported. This commit adds support for
index.warmer.enabled as a dynamic setting.

Closes elastic#3246
spinscale added a commit that referenced this issue Jun 28, 2013
Even though proposed in the documentation, the realtime enabling/disabling of
index warmers was not supported. This commit adds support for
index.warmer.enabled as a dynamic setting.

Closes #3246
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
Even though proposed in the documentation, the realtime enabling/disabling of
index warmers was not supported. This commit adds support for
index.warmer.enabled as a dynamic setting.

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

Successfully merging a pull request may close this issue.

2 participants