lucasbru opened a new pull request, #15445:
URL: https://github.com/apache/kafka/pull/15445

   When the consumer is closed, we perform a sychronous autocommit. We don't 
want to be woken up here, because we are already executing a close operation 
under a deadline. This is in line with the behavior of the old consumer.
   
   This fixes PlaintextConsumerTest.testAutoCommitOnCloseAfterWakeup which is 
flaky on trunk - because we return immediately from the synchronous commit with 
a WakeupException, which causes us to not wait for the commit to finish and 
thereby sometimes miss the committed offset when a new consumer is created.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to