DanielLeens opened a new pull request, #11189:
URL: https://github.com/apache/seatunnel/pull/11189

   ## Why
   
   
`SeaTunnelEngineClusterRoleTest#testWorkerIsFirstMemberThenGetJobDetailStatus` 
can observe `CANCELING` on the client right after `cancelJob` is accepted 
during role failover. The existing assertion expects the client-facing status 
string to become `CANCELED` directly, which makes unrelated PRs fail on shared 
CI.
   
   ## What
   
   - wait for `waitForJobCompleteV2()` to return a terminal `CANCELED` job 
result first
   - assert the client-side job status after the terminal result is available
   - keep the change scoped to the flaky test path only
   
   ## Validation
   
   - `./mvnw spotless:apply`
   - `./mvnw -f seatunnel-engine/seatunnel-engine-client/pom.xml 
-Dtest=SeaTunnelEngineClusterRoleTest#testWorkerIsFirstMemberThenGetJobDetailStatus
 -DfailIfNoTests=false -Dspotless.check.skip=true test`
   
   ## Notes
   
   - A broader `./mvnw -pl seatunnel-engine/seatunnel-engine-client -am ... 
test` attempt on current `dev` is blocked by an unrelated compile issue in 
`seatunnel-config-shade`, so the closest isolated validation was run against 
the exact touched test method.
   


-- 
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]

Reply via email to