pvary commented on PR #16517: URL: https://github.com/apache/iceberg/pull/16517#issuecomment-4515205130
We usually create a single big PR where there are 4 commits: - Rename the current top version to the new one - this helps us keep the history (2.1->2.2) - Copy that version to the original one - this restores the current version (2.2->2.1) - Drop the old version (1.20) - Fix the build, so the new version compiling - we can review the required changes independently We merge these commit without squashing them, but at the same time, so the history is clear. This is a big change, and every other Flink PRs need to pause a bit. Since there is quite a bit of development happening, we might want to coordinate on the dev list. -- 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]
