DanielLeens commented on PR #12109: URL: https://github.com/apache/seatunnel/pull/12109#issuecomment-5617168186
Quick CI-fact update on this same head (`40373e75c`): my last review reported `testManyPipelinesRestoreContentionInWorkerDown` passing on both JDK 8 and JDK 11 in the fork's `engine-v2-it` run, which was the basis for downgrading the stability rating to Stable and recommending merge. A CI rerun on this same head that completed after my review shows `engine-v2-it` failing again on both JDK 8 and JDK 11, with `testManyPipelinesRestoreContentionInWorkerDown` itself as the failure in both jobs this time: `org.awaitility.core.ConditionTimeoutException` at `SplitClusterFaultToleranceIT.java:1601`, "expected: <true> but was: <false> within 5 minutes" - same assertion site, same error shape, on both JDKs. This reopens the question of whether the new test is reliably stable rather than intermittently passing under CI load. Worth a couple more reruns to establish a real pass rate before treating this as ready to merge. -- 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]
