danny0405 commented on code in PR #17987:
URL: https://github.com/apache/hudi/pull/17987#discussion_r2726347152
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/table/action/clean/CleanActionExecutor.java:
##########
@@ -85,6 +85,10 @@ private static boolean deleteFileAndGetResult(HoodieStorage
storage, String dele
} catch (FileNotFoundException fio) {
// With cleanPlan being used for retried cleaning operations, its
possible to clean a file twice
return false;
+ } catch (IOException e) {
+ // Wrap and rethrow
+ log.error("Delete file failed: {}", deletePath, e);
Review Comment:
do we need to check the existence again before throwing?
--
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]
