ahuang98 commented on PR #19428:
URL: https://github.com/apache/kafka/pull/19428#issuecomment-2791164065

   It doesn't seem worth to try removing sessions preemptively, e.g. in 
ReplicationControlManager#processBrokerHeartbeat skipping broker session touch 
if `states.current().equals(CONTROLLED_SHUTDOWN) && 
states.next().equals(SHUTDOWN_NOW)` - imo this is overcomplicated and likely 
more susceptible to bugs


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