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

Elasticsearch bulk error response message support #310

Merged
merged 1 commit into from Nov 17, 2015

Conversation

Projects
None yet
4 participants
@urso
Copy link
Contributor

urso commented Nov 17, 2015

fix parsing of per item error response message from bulk error handling code
supporting both Elasticsearch 1 (unstructured) and 2 (structured) error
reporting.

@urso

This comment has been minimized.

Copy link
Contributor Author

urso commented Nov 17, 2015

closes #309

@tsg

This comment has been minimized.

Copy link
Contributor

tsg commented Nov 17, 2015

LGTM, thanks for the super quick fix. Change log update?

@elasticsearch-release

This comment has been minimized.

Copy link

elasticsearch-release commented Nov 17, 2015

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'.

@andrewkroh

This comment has been minimized.

Copy link
Member

andrewkroh commented Nov 17, 2015

LGTM.

Elasticsearch bulk error response message support
fix parsing of per item error response message from bulk error handling code
supporting both Elasticsearch 1 (unstructured) and 2 (structured) error
reporting.

@urso urso force-pushed the urso:bug/309-es-structured-errors branch from 109146d to 477da12 Nov 17, 2015

andrewkroh added a commit that referenced this pull request Nov 17, 2015

Merge pull request #310 from urso/bug/309-es-structured-errors
Elasticsearch bulk error response message support

@andrewkroh andrewkroh merged commit 5313047 into elastic:1.0.0-rc2 Nov 17, 2015

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
CLA Commit author is a member of Elasticsearch
Details

@urso urso deleted the urso:bug/309-es-structured-errors branch Nov 19, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.