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

Deduplicate field names returned by simpleMatchToFullName & simpleMatchToIndexNames in FieldMappersLookup #11377

Merged
merged 1 commit into from May 28, 2015

Conversation

javanna
Copy link
Member

@javanna javanna commented May 27, 2015

Relates to #10916

@nik9000
Copy link
Member

nik9000 commented May 27, 2015

Makes sense.

@javanna javanna changed the title Internal: deduplicate field names returned by simpleMatchToFullName & simpleMatchToIndexNames in FieldMappersLookup Internal: deduplicate field names returned by simpleMatchToFullName & simpleMatchToIndexNames in FieldMappersLookup May 27, 2015
@javanna javanna force-pushed the enhancement/dedup_field_names branch from 6f88a91 to 32079f2 Compare May 27, 2015 15:02
javanna added a commit to javanna/elasticsearch that referenced this pull request May 27, 2015
… & `simpleMatchToIndexNames` in FieldMappersLookup

Relates to elastic#10916
Closes elastic#11377
@rjernst
Copy link
Member

rjernst commented May 27, 2015

LGTM

… & `simpleMatchToIndexNames` in FieldMappersLookup

Relates to elastic#10916
Closes elastic#11377
@javanna javanna force-pushed the enhancement/dedup_field_names branch from 32079f2 to 2f57ae9 Compare May 28, 2015 07:14
@javanna javanna removed the review label May 28, 2015
@javanna javanna merged commit 2f57ae9 into elastic:master May 28, 2015
@clintongormley clintongormley changed the title Internal: deduplicate field names returned by simpleMatchToFullName & simpleMatchToIndexNames in FieldMappersLookup Deduplicate field names returned by simpleMatchToFullName & simpleMatchToIndexNames in FieldMappersLookup Jun 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants