TSFenwick commented on PR #14776: URL: https://github.com/apache/druid/pull/14776#issuecomment-1670512215
@maytasm @zachjsh i have two thoughts 1) is the retry if it just retries the `AmazonServiceException` and doesn't retry the `MultiObjectDeleteException` that's a benefit compared to the current code. we would just need to comment that it wont retry on `MultiObjectDeleteException` 2) is we are essentially getting a list of all buckets and files that can't be deleted since we are logging them. We could just try to do another delete call on that information. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
