CodeWithPravinMaske commented on PR #12608: URL: https://github.com/apache/seatunnel/pull/12608#issuecomment-6190288454
Thanks @DanielLeens, good catch on the CI budget. Rather than re-running and hoping for a faster runner, I applied your suggested trim. Otherwise every later run, including the one at merge time, would sit at the edge of the 120-minute limit. - `testMysqlCdcNullInNullableUniqueKeyWithoutPrimaryKey` is now disabled on Flink, like the restore test and the other Zeta-only tests in `AbstractMysqlCDCITBase`. The parser override is engine-independent, so the Flink variants added time without covering anything new. - No assertions were dropped. The test still runs on Zeta, in both `MysqlCDCIT` and `Mysql8_4CDCIT`. The `part-7` failure (Maven wrapper `429 Too Many Requests`) will be retried by the new run. -- 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]
