ethan-tyler commented on PR #1987: URL: https://github.com/apache/iceberg-rust/pull/1987#issuecomment-3709100415
> Hi @ethan-tyler , before you digging too deep, here is an exisiting PR(#1606) that covers some changes here already. I'm planning to open an epic issue to track this effort in a more organized fashion Hey @CTTY - thanks for the heads up, I looked at #1606 and there's definitely overlap in the delete file handling. This PR is intentionally focused on just the SnapshotProducer piece with validation. It could serve as a clean base that #1606 rebases onto, and it's intended to unlock the DML actions. Happy to sync on how this fits with your epic and help out where needed. I'm planning to continue with RowDelta, Delete, and Overwrite after this feature lands. Let me know if I should hold off or if landing this first helps. -- 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]
