kinolaev commented on PR #15727: URL: https://github.com/apache/iceberg/pull/15727#issuecomment-5511978361
Since this action no longer depends on Spark, I moved it and its tests to the core module and turned the Spark version into a thin adapter. How does this look to you, @huaxingao? Prior to this, in commit 2285de716e3d05a206987b570c02f124fb043393, I renamed `org.apache.iceberg.actions.DanglingDeleteFiles` to `org.apache.iceberg.util.DanglingDeleteFileUtil`. If you'd prefer to keep only the utility in core rather than the entire action, let me know and I'll drop the latest commit. [Diff before moving the action to core](https://github.com/apache/iceberg/compare/f4db4a56583d268a152a2e725ba4f798f86c4d74..2285de716e3d05a206987b570c02f124fb043393). -- 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]
