kevinjqliu commented on PR #15407: URL: https://github.com/apache/iceberg/pull/15407#issuecomment-6160298966
Thanks for chiming in @anoopj, I think you hit some of the same points I've been thinking about. 1. Iceberg V4 and Delta V5(?) will likely unified the on disk metadata representation 🥳 which makes metadata translation obsolete. 2. UniForm / XTable(?) might handle the translation fairly well already, so we might not need to duplicate the effort here. That being said, i can still see a place for `iceberg-delta-lake` as a migration path for existing Delta tables into the current Iceberg versions (v1, v2, v3), especially for folks who can't or don't want to enable UniForm. @vladislav-sidorovich has put a lot of time and effort into this PR over the past several months, and i want to make sure that work lands somewhere it'll be used and maintained. As i see it, there are 2 paths forward: either we gather signal from the community that the module is useful and agree on its scope together, or we sunset it here in favor of moving this logic to another project. The devlist is a better place to move these discussions. As for this PR, I'm happy to approve after we get rid of the unnecessary test files -- 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]
