Skip to content

Get field mapping documented fields as field#13902

Merged
Mpdreamz merged 1 commit intomasterfrom
fix/get-field-mapping-restspec
Oct 6, 2015
Merged

Get field mapping documented fields as field#13902
Mpdreamz merged 1 commit intomasterfrom
fix/get-field-mapping-restspec

Conversation

@Mpdreamz
Copy link
Copy Markdown
Member

@Mpdreamz Mpdreamz commented Oct 2, 2015

This PR brings this spec more inline with other api's and minimizes the known route variables in the API

See: https://github.com/elastic/elasticsearch/blob/master/core/src/main/java/org/elasticsearch/rest/action/admin/indices/mapping/get/RestGetFieldMappingAction.java#L66

@clintongormley clintongormley added >bug >breaking :Core/Infra/REST API REST infrastructure and utilities labels Oct 2, 2015
@clintongormley
Copy link
Copy Markdown
Contributor

LGTM

Mpdreamz added a commit that referenced this pull request Oct 6, 2015
Get field mapping documented fields as field
@Mpdreamz Mpdreamz merged commit e4049cc into master Oct 6, 2015
@Mpdreamz Mpdreamz deleted the fix/get-field-mapping-restspec branch October 6, 2015 09:26
@Mpdreamz
Copy link
Copy Markdown
Member Author

Mpdreamz commented Oct 6, 2015

merged to master and cherry-picked to 2.0. 2.1, 2.x.

Mpdreamz added a commit to Mpdreamz/elasticsearch that referenced this pull request Oct 6, 2015
After `field` has been renamed to `fields` in elastic#13902
Mpdreamz added a commit that referenced this pull request Oct 6, 2015
After `field` has been renamed to `fields` in #13902
Mpdreamz added a commit that referenced this pull request Oct 6, 2015
After `field` has been renamed to `fields` in #13902
Mpdreamz added a commit that referenced this pull request Oct 6, 2015
After `field` has been renamed to `fields` in #13902
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>breaking >bug :Core/Infra/REST API REST infrastructure and utilities

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants