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 AbstractFieldMapper #12089

Merged
merged 1 commit into from Jul 8, 2015

Conversation

@rjernst
Copy link
Member

commented Jul 7, 2015

AbstractFieldMapper is the only direct base class of FieldMapper.
This change moves all AbstractFieldMapper functionality into
FieldMapper, since there is no need for 2 levels of abstraction.

AbstractFieldMapper is the only direct base class of FieldMapper.
This change moves all AbstractFieldMapper functionality into
FieldMapper, since there is no need for 2 levels of abstraction.
@jpountz

This comment has been minimized.

Copy link
Contributor

commented Jul 7, 2015

LGTM

rjernst added a commit that referenced this pull request Jul 8, 2015
@rjernst rjernst merged commit 8d9053a into elastic:master Jul 8, 2015
1 check passed
1 check passed
CLA Commit author is a member of Elasticsearch
Details
@rjernst rjernst deleted the rjernst:refactor/field-mapper-collapse branch Jul 8, 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.