MartijnVisser commented on PR #29301: URL: https://github.com/apache/flink/pull/29301#issuecomment-5871011397
@snuyanzin I've pushed the `runTest` change to https://github.com/MartijnVisser/flink/commits/FLINK-34427-runtest-wait-for-close. Pinned to one CPU, the branch had 0 crashes in 800 class runs, against 12 in 498 on `master`. With a `requestSlot` that blocks until the executor is shut down (at most 2 s), `master` and this PR crash every time and the branch never does. Could you swap the commit here for that one? The cancel isn't needed for this ticket, so I'd leave it out. -- 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]
