gnodet opened a new pull request, #24626: URL: https://github.com/apache/camel/pull/24626
## Summary _Claude Code on behalf of gnodet_ Fix flaky `InOutQueueProducerAsyncLoadTest` by reducing the message count and improving synchronization. **Root cause**: The test sent 5000 InOut JMS request-reply messages via a 2-thread executor with only a 30-second Awaitility timeout. On slow CI machines, processing 5000 round-trip messages through an embedded ActiveMQ broker frequently exceeded the timeout. Additionally, test failures inside the executor threads were silently swallowed. **Fix**: - Reduce message count from 5000 to 500 (still validates concurrent InOut load behavior) - Replace `Awaitility.await().until(executor::isTerminated)` with a `CountDownLatch` for precise completion tracking - Add `AtomicInteger` to count and assert zero failures (previously swallowed) - Use `executor.awaitTermination()` with explicit 10s timeout instead of polling - Increase overall completion timeout to 60 seconds - Modernize anonymous `Runnable` to lambda ## Test plan - [x] `InOutQueueProducerAsyncLoadTest` passes locally (1 test, 0 failures, 4.7s) - [ ] CI build passes 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.6 <[email protected]> -- 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]
