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

Add options for indices.get feature #13946

Merged
merged 1 commit into from
Oct 6, 2015
Merged

Add options for indices.get feature #13946

merged 1 commit into from
Oct 6, 2015

Conversation

gmarz
Copy link
Contributor

@gmarz gmarz commented Oct 5, 2015

@gmarz gmarz added the :Core/Infra/REST API REST infrastructure and utilities label Oct 5, 2015
@nik9000
Copy link
Member

nik9000 commented Oct 5, 2015

LGTM

@clintongormley
Copy link
Contributor

Hmmm doesn't this clash with eg the GET index/_settings API etc?

@gmarz
Copy link
Contributor Author

gmarz commented Oct 6, 2015

I think the difference is that indices.get allows you to retrieve multiple features (_settings, _aliases, etc..) all in the same call, which isn't possible with the other APIs since they are feature specific.

@clintongormley
Copy link
Contributor

ah right, i wasn't aware of that!

gmarz added a commit that referenced this pull request Oct 6, 2015
@gmarz gmarz merged commit 9200c44 into master Oct 6, 2015
@gmarz gmarz deleted the fix/indices-get-restspec branch October 6, 2015 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/REST API REST infrastructure and utilities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants