Skip to content

Conversation

@mb1337
Copy link

@mb1337 mb1337 commented Jul 17, 2019

BulkAll will hang indefinitely, giving no error, if a bulk request keeps returning BadResponse. The bulk all request should throw an error after hitting the back off retry number.

BulkAll will hang indefinitely, giving no error, if a bulk request keeps coming back as BadResponse.
@russcam
Copy link
Contributor

russcam commented Jul 31, 2019

Thanks for opening a PR, @mb1337. Do you have a reproducible test to add, that demonstrates the issue you've seen with continually returned BadResponse from BulkAll?

Mpdreamz added a commit that referenced this pull request Aug 9, 2019
@Mpdreamz Mpdreamz mentioned this pull request Aug 9, 2019
@Mpdreamz
Copy link
Member

Mpdreamz commented Aug 9, 2019

Thanks @mb1337 for opening this and bringing this to our attention.

I spent some time reproducing this in a test and opened #4014 to fix this and took the time to introduce some other improvements too.

Closing this in favour of #4014

@Mpdreamz Mpdreamz closed this Aug 9, 2019
codebrain pushed a commit that referenced this pull request Aug 12, 2019
…equest keeps returning a bad statuscode (#4014)

Ensure BulkAll() is aborted properly if the whole bulk request keeps returning a bad statuscode.
codebrain pushed a commit that referenced this pull request Aug 12, 2019
…equest keeps returning a bad statuscode (#4014)

Ensure BulkAll() is aborted properly if the whole bulk request keeps returning a bad statuscode.
codebrain pushed a commit that referenced this pull request Aug 12, 2019
…equest keeps returning a bad statuscode (#4014)

Ensure BulkAll() is aborted properly if the whole bulk request keeps returning a bad statuscode.
russcam pushed a commit that referenced this pull request Aug 13, 2019
…equest keeps returning a bad statuscode (#4014)

Ensure BulkAll() is aborted properly if the whole bulk request keeps returning a bad statuscode.

(cherry picked from commit c0293a0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants