wombatu-kun opened a new pull request, #8736: URL: https://github.com/apache/paimon/pull/8736
### Purpose `MySqlSyncDatabaseActionITCase.testSyncMultipleShards` intermittently fails CI with `java.util.concurrent.TimeoutException: testSyncMultipleShards() timed out after 120 seconds` (for example, it failed the CI run of an unrelated PR #8735, whose changes are confined to `paimon-common` global btree index code and cannot reach Flink CDC). This is a flaky timeout, not a functional regression. Two things make the test prone to spurious timeouts. First, the 120s per-method budget is tight for this test. Unlike its siblings it does several sequential `waitForResult` phases, each preceded by a schema `ALTER` that forces the CDC source to re-snapshot the changed table, on top of Flink job startup and the initial MySQL snapshot/binlog switchover. Under a loaded CI runner this legitimately exceeds 120s even though the data volume is tiny. This raises the timeout to 240s, the same value already used by the comparable heavy multi-table test in this class, `testSyncManyTableWithLimitedMemory` (the newly-added-table tests use 600s). Second, the test called the `JobClient`-less `waitForResult` overload, in which `checkJobNotTerminated(...)` is a no-op. If the CDC job ever dies, the test cannot tell a dead job from a slow one and simply hangs until the outer `@Timeout` fires, producing an opaque timeout with no diagnosis. This captures the `JobClient` returned by `runActionWithDefaultEnv(...)` and threads it through the four `waitForResult` calls, so a terminated job now fails fast with a clear `AssertionError` instead of hanging. This matches the pattern already used by other tests in the same file (for example the newly-added-table tests). No production code changes; test-only. ### Tests `MySqlSyncDatabaseActionITCase.testSyncMultipleShards` (the modified test itself). -- 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]
