wirybeaver commented on issue #2269: URL: https://github.com/apache/iceberg-rust/issues/2269#issuecomment-4105133520
@Shekharrajak Appreciate collecting all related issues. To keep the momentum going for both `merge_into` and the `update` DataFusion integration, I’d suggest we decouple the basic RowDeltaAction (CoW) from the more complex MoR implementation. I have the CoW implementation ready and tested it together with for my merge_into full PoC https://github.com/apache/iceberg-rust/issues/2201 and update full PoC https://github.com/apache/iceberg-rust/issues/2205, merging the "bare" RowDeltaAction now would: - Unblock downstream PRs for merge_into and update immediately. - Reduce the scope of the MoR PR, making it easier and faster for maintainers to review. The author of https://github.com/apache/iceberg-rust/issues/2218 can rebase on my implementation for the more complex MoR. -- 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]
