[GitHub] [incubator-opendal] nasen23 opened a new pull request, #1840: feat: Retry when some of batch operations are failing

["nasen23 (via GitHub)" <gi...@apache.org>]

nasen23 opened a new pull request, #1840:
URL: https://github.com/apache/incubator-opendal/pull/1840

   Also added a test for retry on batch operation.
   
   This implementation works as expected but is not so elegant as far as I'm considering. Batch operation could possibly be retried even when result is `Ok`. So I had to wrap it in a `Result<(), Result<RpBatch>>` type. And the `errors` method on `BatchedResults` requires dynamic dispatching.
   
   Regarding the issue mentioned above, would you be able to suggest any better solutions or provide further guidance?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-opendal] Xuanwo merged pull request #1840: feat: Retry when some of batch operations are failing

["Xuanwo (via GitHub)" <gi...@apache.org>]

Xuanwo merged PR #1840:
URL: https://github.com/apache/incubator-opendal/pull/1840


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org