pvary commented on PR #6222: URL: https://github.com/apache/iceberg/pull/6222#issuecomment-1336098519
Missclicked 😢 What I was trying to comment: Sadly I don't have enough time to properly review the PR, but I remember that we had issues with the partition evolution in the past when @szlta tested the Hive metadata tables. IIRC the issues surfaced when we changed from (a, b) partitioning to (a, c). See the relevant PRs: https://github.com/apache/iceberg/pulls?q=is%3Apr+szlta+partition The issues are handled on Core level, but it might worth to add some checks for edge cases too. Like: - Removed columns - Readded columns - Partition evolution - Schema evolution -- 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
