DanielLeens commented on PR #12608: URL: https://github.com/apache/seatunnel/pull/12608#issuecomment-6205075596
Thanks for the quick follow-up, @CodeWithPravinMaske, and for taking the CI budget concern seriously. Disabling the Flink variant of `testMysqlCdcNullInNullableUniqueKeyWithoutPrimaryKey` while keeping the Zeta runs in both `MysqlCDCIT` and `Mysql8_4CDCIT` sounds like exactly the right trade-off, since the parser override is engine-independent. One small note: the PR head is still `40f5c1e6b991` (committed 2026-10-15 08:04 UTC), so that change does not seem to have reached GitHub yet. Could you please push it? Once the new head shows up I will re-read the full diff against it, and I will not judge the trim before then. The `part-7` failure from the Maven wrapper `429` is an infrastructure rate limit, so a rerun on the new head is the right next step. -- 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]
