prashantwason commented on PR #18035: URL: https://github.com/apache/hudi/pull/18035#issuecomment-3837690577
Thanks for the review! You're right - since clean execution fails at L79 if deletion returns `false`, by the time we reach MDT to apply a clean commit, all files are guaranteed to be deleted or non-existent. I've updated the PR to remove the `HoodieTableMetadataUtil` changes and the associated test. The fix now only includes returning `true` for `FileNotFoundException` in `CleanActionExecutor`, which is sufficient to handle the case where a file has already been deleted in a prior retry. -- 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]
