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

   The contract of KafkaConsumer.poll(Duration) says that it throws 
InterruptException "if the calling thread is interrupted before or while this 
function is called". The new KafkaConsumer implementation was not doing this if 
the thread was interrupted before the poll was called, specifically with a very 
short timeout. If it ever waited for records, it did check the thread state. If 
it did not wait for records because of a short timeout, it did not.
   
   Some of the log messages in the code were erroneously mentioned timeouts, 
when they really meant interruption.
   
   Also adds a test for this specific scenario.
   
   ### 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