DanielLeens opened a new pull request, #11236: URL: https://github.com/apache/seatunnel/pull/11236
### Purpose Stabilize SplitClusterPendingJobLifecycleFailoverIT.testPendingJobLifecycleInMasterFailover after CI observed the job remain in CANCELING for two minutes. ### Root cause After master failover, the top-level job status can become RUNNING before every restored pipeline and task vertex has reacquired slots and reported RUNNING. Cancelling immediately after the top-level state check races with restore and resource scheduling, so the assertion can observe CANCELING instead of the final CANCELED state. ### Changes - Wait for the active master restored physical plan to have all pipelines and physical or coordinator vertices in RUNNING state before issuing cancelJob. - Keep the assertion focused on the pending-job lifecycle transition without changing production cancellation logic. ### Validation - ./mvnw -pl seatunnel-e2e/seatunnel-engine-e2e/connector-seatunnel-e2e-base spotless:apply -nsu -Dmaven.gitcommitid.skip=true - JAVA_HOME=/Library/Java/JavaVirtualMachines/jdk1.8.0_171.jdk/Contents/Home ./mvnw -B -pl :connector-seatunnel-e2e-base -DskipTests -DskipITs=true -Dlicense.skipAddThirdParty=true -Dskip.ui=true -Dskip.spotless=true -Dmaven.gitcommitid.skip=true -nsu test-compile - JAVA_HOME=/Library/Java/JavaVirtualMachines/jdk1.8.0_171.jdk/Contents/Home ./mvnw -B -pl :connector-seatunnel-e2e-base -DskipITs=false -DskipTests=false -Dtest=SplitClusterPendingJobLifecycleFailoverIT#testPendingJobLifecycleInMasterFailover -Dmaven.test.dependency.excludes=org.apache.seatunnel:seatunnel-flink-starter-common,org.apache.seatunnel:seatunnel-flink-13-starter,org.apache.seatunnel:seatunnel-flink-15-starter,org.apache.seatunnel:seatunnel-flink-20-starter,org.apache.seatunnel:seatunnel-spark-starter-common,org.apache.seatunnel:seatunnel-spark-2-starter,org.apache.seatunnel:seatunnel-spark-3-starter -Dlicense.skipAddThirdParty=true -Dskip.ui=true -Dskip.spotless=true -Dmaven.gitcommitid.skip=true -nsu clean test seatunnel-engine-ui was not modified. It was not revalidated directly beyond the Maven dependency path used by the engine E2E test. -- 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]
