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.

Sign up for GitHub

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

Jump to bottom

[Transform] Handle bulk indexing errors due to mapping conflicts #50122

Closed
hendrikmuhs opened this issue Dec 12, 2019 · 1 comment · Fixed by #51307
Closed

[Transform] Handle bulk indexing errors due to mapping conflicts #50122

hendrikmuhs opened this issue Dec 12, 2019 · 1 comment · Fixed by #51307
Labels
>enhancement :ml/Transform Transform

Comments

@hendrikmuhs
Copy link

If a transform runs into a bulk indexing failure it handles the failure as non-persistent except for:

Mapping conflicts should be added to these checks.
@elasticmachine
Copy link
Collaborator

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

@hendrikmuhs hendrikmuhs mentioned this issue Dec 12, 2019
Closed
@hendrikmuhs hendrikmuhs mentioned this issue Jan 22, 2020
Merged
hendrikmuhs pushed a commit to hendrikmuhs/elasticsearch that referenced this issue Jan 22, 2020
check bulk indexing error for permanent problems
d2b17bf 
fixes elastic#50122
hendrikmuhs pushed a commit that referenced this issue Jan 23, 2020
[Transform] Handle permanent bulk indexing errors (#51307)
589e033 
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
hendrikmuhs pushed a commit that referenced this issue Jan 23, 2020
[Transform] Handle permanent bulk indexing errors (#51307)
3553f68 
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
@codebrain codebrain mentioned this issue Apr 1, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
>enhancement :ml/Transform Transform
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Transform] Handle permanent bulk indexing errors hendrikmuhs/elasticsearch
2 participants
@hendrikmuhs @elasticmachine