chia7712 commented on PR #16686:
URL: https://github.com/apache/kafka/pull/16686#issuecomment-2413455632

   > Then, if you try to close the consumer or to unsubscribe to cleanly leave 
the group, the old consumer works because there are no interrupt checks on 
those paths whereas the new one fails short because calling get() (e.g. on a 
future) will thrown an interrupt exception again. This is where the difference 
is.
   
   Yes, that’s the side effect when we delegate all tasks to the background 
thread and then block on wait—making many async consumer APIs interruptible.
   
   Do we have a rulebook on how consumer APIs should handle interruptions if 
we’re aiming to align behavior? If not, maybe we need to draw these first. In 
this PR, we’re juggling two different interruption issues:
   
        1.      Interruptions can disrupt the execution of listeners, close, 
and unsubscribe.
        2.      Interruptions aren’t being propagated to the listener or 
background thread.
   
   


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