liurenjie1024 commented on PR #1455: URL: https://github.com/apache/iceberg-rust/pull/1455#issuecomment-2993575061
Hi, @cmcarthur This pr is too large to review. I would suggest to split them into several small prs, for example, the `ExpireSnapshotAction` could be a good start. Also as mentioned in https://github.com/apache/iceberg-rust/issues/1453#issuecomment-2993572461, I don't think it's a good idea to put procedure into core library. You could try to extend datafusion to do this. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
