Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Catch BulkIndexError exception in async_bulk. resolves #864 #2208

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bshakur8
Copy link

fix #864

async_bulk can raise BulkIndexError despite setting raise_on_error=False, something that prevents the rest of the bulk from running. Besides, as mentioned in #864 it prevents using additional flags like max_retries.

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@cla-checker-service
Copy link

❌ Author of the following commits did not sign a Contributor Agreement:
d71e651

Please, read and sign the above mentioned agreement if you want to contribute to this project

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

streaming_bulk fails to retry if max_retries is set but raise_on_error is kept on True
3 participants