joyhaldar commented on PR #16097: URL: https://github.com/apache/iceberg/pull/16097#issuecomment-4326229007
> @joyhaldar Thanks for the update. > > The usual workflow is to merge the changes for the newest Flink version first, then backport to the older versions. It's starting to get hard to review the changes. Could you remove all changes apart from the Flink 2.1 changes? Done, thank you for the guidance. Scoped to Flink 2.1 only. Also reverted the change to `DataTestBase.SUPPORTED_PRIMITIVES` for now, since adding `UUID` there would regress `v2.0` and `v1.20` until they receive the same fixes. If this PR is accepted, I will follow up with backport PRs for `v2.0` and `v1.20`, and then a separate PR to add `UUID` to `SUPPORTED_PRIMITIVES`. Sorry about the noise on the earlier scope, I should have started narrower. -- 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]
