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

Remove fieldSearchAnalyzer and fieldSearchQuoteAnalyzer from MapperService. #9262

Merged
merged 1 commit into from Jan 12, 2015

Conversation

@rjernst
Copy link
Member

commented Jan 12, 2015

Instead, get the FieldMapper for the field and check the analyzer there. There was only one use of fieldSearchAnalyzer and no uses of the quoted version.

@jpountz

This comment has been minimized.

Copy link
Contributor

commented Jan 12, 2015

LGTM

MapperService.

Instead, get the FieldMapper for the field and check the analyzer there.
@rjernst rjernst force-pushed the rjernst:remove/fieldSearchAnalyzer branch to 48bc132 Jan 12, 2015
@rjernst rjernst removed the review label Jan 12, 2015
@rjernst rjernst merged commit 48bc132 into elastic:master Jan 12, 2015
1 check passed
1 check passed
CLA Commit author is a member of Elasticsearch
Details
@rjernst rjernst deleted the rjernst:remove/fieldSearchAnalyzer branch Jan 21, 2015
@clintongormley clintongormley changed the title Mappings: Remove fieldSearchAnalyzer and fieldSearchQuoteAnalyzer from MapperService. Remove fieldSearchAnalyzer and fieldSearchQuoteAnalyzer from MapperService. Jun 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.