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

[Transform] Handle permanent bulk indexing errors #51307

Merged

Conversation

hendrikmuhs
Copy link
Contributor

check bulk indexing error for permanent problems and ensure the state goes into failed instead of retry. Corrects the stats API to show the real error and avoids excessive audit logging.

fixes #50122

@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml/Transform)

@hendrikmuhs hendrikmuhs force-pushed the transform-bulkindex-error-#50122 branch from a887d8f to cefa7b3 Compare January 22, 2020 13:48
@benwtrent benwtrent self-requested a review January 22, 2020 14:05
@hendrikmuhs
Copy link
Contributor Author

run elasticsearch-ci/2

@hendrikmuhs hendrikmuhs merged commit 589e033 into elastic:master Jan 23, 2020
hendrikmuhs pushed a commit that referenced this pull request Jan 23, 2020
check bulk indexing error for permanent problems and ensure the state goes into failed instead of
retry. Corrects the stats API to show the real error and avoids excessive audit logging.

fixes #50122
debadair pushed a commit to debadair/elasticsearch that referenced this pull request Jan 28, 2020
check bulk indexing error for permanent problems and ensure the state goes into failed instead of
retry. Corrects the stats API to show the real error and avoids excessive audit logging.

fixes elastic#50122
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.

[Transform] Handle bulk indexing errors due to mapping conflicts
4 participants