poorbarcode opened a new pull request, #17837:
URL: https://github.com/apache/pulsar/pull/17837

   Fixes:
   
   -#14109
   
   
   ### Motivation
   
   The expected execution flow for this test is: 
   
   1. send 505 messages
   2. commit 10 transactions, every transaction ack 50 messages
   3. receive the last 5 messages in the last transaction, wait for transaction 
timeout
   4. confirm that the last 5 messages can be consumed by new consumer
   
   <strong>(High light)</strong> The default value for transaction TTL is 10 
seconds, and the default value for `Awaitility.await` is also 10 seconds,  so 
this test is not stable.
   
   Note: This is a guess cause, the problem is not reproduced locally, but 
after TTL is set to 11s, the probability of the problem occurring is 100%.
   
   ### Modifications
   
   Fix flaky test
   - set transaction TTL to 5s
   
   Other changes
   - define a name for the task thread
   - acknowledge the last 5 messages
   
   ### Documentation
   
   - [ ] `doc-required` 
   
   - [ ] `doc-not-needed` 
   
   - [ ] `doc` 
   
   - [ ] `doc-complete`
   
   ### Matching PR in forked repository
   
   PR in forked repository: 
   
   - https://github.com/poorbarcode/pulsar/pull/13
   


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