MartijnVisser opened a new pull request, #29403: URL: https://github.com/apache/flink/pull/29403
Backport of #29386, which is approved but not merged on master yet. `getRunningFetcher()` has the same `isEmpty()`-then-`next()` race on release-1.20, so `addSplits()` can fail with `NoSuchElementException` when a fetcher removes itself in between. The only difference is that the test method is `public`, since this test class is still JUnit 4 on release-1.20. `testGetRunningFetcherSurvivesFetcherRemovalInToctouWindow` fails without the fix, with the ticket's `NoSuchElementException`, and passes with it on JDK 8 and 11. `SplitFetcherManagerTest`, `spotless:check` and `checkstyle:check` pass. ##### 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]
