SEPURI-SAI-KRISHNA commented on PR #12495:
URL: https://github.com/apache/seatunnel/pull/12495#issuecomment-5882872083
CI update on the current head `ade4e41c86`, since the previous run's picture
has changed a lot and the one remaining red is now precisely attributable.
The run is 81 success, 1 failure, 1 cancelled, down from 7 failures. In
particular **all four unit lanes now pass, including `unit-test (11,
windows-latest)`**, and `Code style` passes. That is #12444 landing: merging
current `dev` into this branch picked up the `PayPalClientTest` latch fix, and
the Windows flake that reddened the previous two runs is gone.
The single remaining failure is `engine-v2-it (8, ubuntu-latest)`: 205
tests, 1 error.
```
SplitClusterFaultToleranceIT.testStreamJobCancelResolvesWhenWorkerCrashesBeforeCancelAck
org.awaitility.core.ConditionTimeoutException
Caused by: org.opentest4j.AssertionFailedError: expected: <CANCELED> but
was: <FAILED>
at
SplitClusterFaultToleranceIT.assertEventuallyCanceled(SplitClusterFaultToleranceIT.java:557)
```
That is #12311, "[Fix][Zeta] Resolve a CANCELING vertex to CANCELED, not
FAILED, when its worker is lost", which names this exact test and is already
approved. The JDK 8 leg is where @DanielLeens measured it failing most often.
Nothing in this diff reaches `PhysicalVertex` or the cancel path, and the
`kudu-connector-it (11)` lane next to it was cancelled rather than failed.
So the remaining red here clears when #12311 merges, and needs no change on
this PR.
--
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]
