1996fanrui opened a new pull request, #29867: URL: https://github.com/apache/flink/pull/29867
## What is the purpose of the change Backport of #29278 to release-1.20. ## Verifying this change - `./mvnw -pl flink-runtime -am -Pskip-webui-build -Dtest='DefaultSchedulerTest' -Dsurefire.failIfNoSpecifiedTests=false package` (JDK 11), repeated 5 times: `BUILD SUCCESS` each time, `Tests run: 51, Failures: 0, Errors: 0, Skipped: 0 -- in org.apache.flink.runtime.scheduler.DefaultSchedulerTest` - `./mvnw -q -pl flink-runtime spotless:check checkstyle:check` (JDK 11): passed ## Difference from the original | Original commit | Conflict | Byte-identical | Changes | | --- | --- | --- | --- | | 9412b866063 | No | Yes | None | ## Notes - The Jira reports failures on master, release-2.3 and release-2.2 only. release-1.20 has the same race: `Execution#deploy()` runs `submitTask` on the future executor and hands back via `whenCompleteAsync(..., jobMasterMainThreadExecutor)`, and `ComponentMainThreadExecutorServiceAdapter.forMainThread()` asserts the calling thread in `execute`, while `DefaultSchedulerTest` passes the real `scheduledExecutorService` as future executor. On this branch the TDD is created on the main thread, so only the submission runs on the future executor. ##### Was generative AI tooling used to co-author this PR? - [X] Yes (please specify the tool below) Generated-by: Claude Code (Claude Opus 5.5) -- 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]
