zhangjun0x01 commented on pull request #1715: URL: https://github.com/apache/iceberg/pull/1715#issuecomment-721659857
> @zhangjun0x01 You may want to take a look at this, especially if you have an Idea on how to make the Spark Actions all extend BaseSparkAction I extracted getManifestListPathsăgetOtherMetadataFilePaths and other methods from spark to BaseAction. I think if we do RemoveOrphanFilesAction and ExpireSnapshotsAction for flink in the future, we can use these methods. My idea is to put the common part of spark actions into BaseSparkAction, and put the common part of spark and flink into BaseAction ---------------------------------------------------------------- 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]
