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

Cluster State API: Remove index template filtering #4954

Closed
spinscale opened this Issue Jan 30, 2014 · 1 comment

Comments

Projects
None yet
3 participants
@spinscale
Member

spinscale commented Jan 30, 2014

As the get index template API does a pretty good job of selecting and showing specific index templates to the user, we could potentially remove this functionality, when using the cluster state API - as this functionality is no more needed at that place.

Up for discussion.

@kevinkluge kevinkluge added v1.1.0 and removed v1.0.0 labels Feb 10, 2014

@s1monw s1monw added v1.2.0 and removed v1.1.0 labels Mar 20, 2014

@s1monw

This comment has been minimized.

Contributor

s1monw commented Mar 20, 2014

@spinscale I pushed this to 1.2 lets discuss soon

@spinscale spinscale self-assigned this May 2, 2014

spinscale added a commit to spinscale/elasticsearch that referenced this issue May 5, 2014

Cluster State API: Remove index template filtering
The possibility of filtering for index templates in the cluster state API
had been introduced before there was a dedicated index templates API. This
commit removes this support from the cluster state API, as it was not really
clean, requiring you to specify the metadata and the index templates.

Closes elastic#4954

spinscale added a commit that referenced this issue May 5, 2014

Cluster State API: Remove index template filtering
The possibility of filtering for index templates in the cluster state API
had been introduced before there was a dedicated index templates API. This
commit removes this support from the cluster state API, as it was not really
clean, requiring you to specify the metadata and the index templates.

Closes #4954

@spinscale spinscale added the breaking label May 6, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment