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

Fix doc parser to still pre/post process metadata fields on disabled type #13137

Merged
merged 1 commit into from Aug 30, 2015

Conversation

Projects
None yet
3 participants
@rjernst
Member

rjernst commented Aug 27, 2015

I improved the tests more so we won't hit this problem again.

closes #13017

@tlrx

This comment has been minimized.

Show comment
Hide comment
@tlrx

tlrx Aug 27, 2015

Member

LGTM thanks @rjernst

Member

tlrx commented Aug 27, 2015

LGTM thanks @rjernst

@clintongormley clintongormley added the >bug label Aug 27, 2015

rjernst added a commit that referenced this pull request Aug 30, 2015

Merge pull request #13137 from rjernst/empty_doc_again
Fix doc parser to still pre/post process metadata fields on disabled type

@rjernst rjernst merged commit 2539b77 into elastic:master Aug 30, 2015

1 check passed

CLA Commit author is a member of Elasticsearch
Details

@rjernst rjernst deleted the rjernst:empty_doc_again branch Aug 30, 2015

rjernst added a commit that referenced this pull request Aug 30, 2015

@tlrx

This comment has been minimized.

Show comment
Hide comment
@tlrx

tlrx Aug 31, 2015

Member

@rjernst thanks !

Member

tlrx commented Aug 31, 2015

@rjernst thanks !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment