zhangdove edited a comment on pull request #1313: URL: https://github.com/apache/iceberg/pull/1313#issuecomment-671698264
@RussellSpitzer Thanks for your review that has benefited me a lot. @aokolnychyi As you said, maybe I don't use it in a reasonable way. It wasn't until I saw this PR [#1244](https://github.com/apache/iceberg/pull/1244) that I thought of moving the file deletion method to this method(`removeOrphanFiles()`), so that I later made some other attempts to move the file deletion from driver to Executor. Thank you for your Suggestions. I'm going to close this PR, and I'm going to keep watching and using master branch code ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
