prodeezy opened a new pull request #928: Add failure handling in cleanup while reading snapshot manifest-list files URL: https://github.com/apache/incubator-iceberg/pull/928 https://github.com/apache/incubator-iceberg/issues/822 reported a gap in failure handling during cleanup. If any snapshot's manifest-list file is unreadable/has temporary io issues none of the snapshots from then would be cleaned up. This adds `Tasks` based handling where even if one of them fails the rest are still tried. More context: the try-with-resource block doesn't handle it coz `AvroIterable` throws a `RuntimeIOException` which bypasses any `IOException` handling up the trace. `Tasks`handles this and lets other snapshots have a chance of cleanup. cc @mehtaashish23
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
