gbhavya07 opened a new pull request, #24629:
URL: https://github.com/apache/camel/pull/24629

   # Description
   
   This PR addresses **CAMEL-24024** by improving the reliability of 
`InOutQueueProducerAsyncLoadTest`, which was reported as flaky in CI due to 
high load and timeout-based synchronization.
   
   Previously, the test submitted 5000 asynchronous requests using a 2-thread 
executor and relied on Awaitility to wait for executor termination. Under 
resource-constrained CI environments, the large workload could exceed the 
30-second timeout, resulting in intermittent failures.
   
   To make the test more deterministic while preserving its intent, this change:
   
   - Reduces the message count from 5000 to 500, which is sufficient to 
validate the asynchronous InOut request/reply behavior.
   - Replaces Awaitility-based polling with a `CountDownLatch` to reliably 
track completion of all submitted tasks.
   - Introduces an `AtomicInteger` to track asynchronous request failures and 
verifies that all requests complete successfully.
   
   These changes preserve the purpose of the test while reducing its 
susceptibility to CI timing and resource constraints.
   
   # Target
   
   - [x] I checked that the commit is targeting the correct branch (Camel 4 
uses the `main` branch)
   
   # Tracking
   
   - [x] If this is a large change, bug fix, or code improvement, I checked 
there is a JIRA issue filed for the change (**CAMEL-24024**).
   
   # Apache Camel coding standards and style
   
   - [x] I checked that each commit in the pull request has a meaningful 
subject line and body.
   
   - [x] I have run `mvn clean install -DskipTests` locally from root folder 
and I have committed all auto-generated changes.
   I ran mvn clean install -DskipTests from the project root. The build reached 
camel-kserve and failed due to an unrelated protobuf-maven-plugin/protoc 
generation error. The modified camel-sjms module builds successfully.
   
   # AI-assisted contributions
   
   - [ ] If this PR includes AI-generated code, commits have proper 
co-authorship attribution (e.g., `Co-authored-by` trailers) and the PR 
description identifies the AI tool used.
   
   


-- 
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]

Reply via email to