Skip to content

Make index level mapping apis use MappedFieldType#11559

Merged
rjernst merged 1 commit intoelastic:masterfrom
rjernst:pr/field-type-users
Jun 9, 2015
Merged

Make index level mapping apis use MappedFieldType#11559
rjernst merged 1 commit intoelastic:masterfrom
rjernst:pr/field-type-users

Conversation

@rjernst
Copy link
Copy Markdown
Member

@rjernst rjernst commented Jun 9, 2015

The MapperService is the "index wide view" of mappings. Methods on it
are used at query time to lookup how to query a field. This
change reduces the exposed api so that any information returned
is limited to that api exposed by MappedFieldType. In the future,
MappedFieldType will be guaranteed to be the same across all
document types for a given field.

Note CompletionFieldType needed some more settings moved to it. Other
than that, this change is almost purely cosmetic.

@rjernst rjernst added v2.0.0-beta1 :Search Foundations/Mapping Index mappings, including merging and defining field types labels Jun 9, 2015
@rjernst rjernst changed the title Mappings: Make index level mapping apis to use MappedFieldType Mappings: Make index level mapping apis use MappedFieldType Jun 9, 2015
The MapperService is the "index wide view" of mappings. Methods on it
are used at query time to lookup how to query a field. This
change reduces the exposed api so that any information returned
is limited to that api exposed by MappedFieldType. In the future,
MappedFieldType will be guaranteed to be the same across all
document types for a given field.

Note CompletionFieldType needed some more settings moved to it. Other
than that, this change is almost purely cosmetic.
@rjernst rjernst force-pushed the pr/field-type-users branch from 21c9d51 to 1fdae75 Compare June 9, 2015 17:04
@jpountz
Copy link
Copy Markdown
Contributor

jpountz commented Jun 9, 2015

LGTM

rjernst added a commit that referenced this pull request Jun 9, 2015
Mappings: Make index level mapping apis use MappedFieldType
@rjernst rjernst merged commit 9fa69bd into elastic:master Jun 9, 2015
@rjernst rjernst deleted the pr/field-type-users branch June 9, 2015 18:04
@clintongormley clintongormley changed the title Mappings: Make index level mapping apis use MappedFieldType Make index level mapping apis use MappedFieldType Jun 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>enhancement :Search Foundations/Mapping Index mappings, including merging and defining field types v2.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants