damccorm opened a new pull request, #41116:
URL: https://github.com/apache/beam/pull/41116

   ### Summary
   In `PreCommit Java Jms IO Direct` 
(https://github.com/apache/beam/actions/runs/42628668151), 
`JmsIOIT.testPublishingThenReadingAllIndividualAcknowledge[with client class 
class org.apache.qpid.jms.JmsConnectionFactory]` failed with `AssertionError: 
All messages should be acknowledged expected:<0> but was:<10000>` because the 
30-second default `readTimeout` in `JmsIOIT.runPublishingThenReadingAll` 
expired under CI load and cancelled the read pipeline before Qpid AMQP finished 
reading and acknowledging all 10,000 messages.
   
   This change increases the default `readTimeout` in `JmsIOITOptions` from 30s 
to 60s. Note that `runPublishingThenReadingAll` still exits immediately as soon 
as all 10,000 messages are read and acknowledged (`unackRecords == 0`).
   
   ### Verification
   - Triggered workflow run on branch: 
https://github.com/apache/beam/actions/runs/42651530009


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