gnodet commented on PR #27579: URL: https://github.com/apache/camel/pull/27579#issuecomment-6130999576
@oscerd your CHANGES_REQUESTED review on `dc96094c` flagged deterministic test failures in `DefaultExecutorServiceManagerTest` caused by `WORKER_THREAD_POOL` creating non-daemon threads. That blocking issue is now resolved: the PR switches to `FORK_JOIN_POOL`, which reuses the existing `ForkJoinPool` workers (always daemon), so the thread-factory listener tests pass unchanged. CI is green on both the Java 17 and Java 25 build jobs. Could you re-review or dismiss your change request? Everything from your earlier review has been addressed. -- 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]
