CalvinConfluent commented on PR #18277:
URL: https://github.com/apache/kafka/pull/18277#issuecomment-2626049014

   @junrao @splett2 
   Based on the feedback, I updated the following:
   1. Instead of modifying the whole ClusterControl register logic, I added a 
clean shutdown flag for the ReplicationControl to update the ISR and ELR 
accordingly.
   
   - If it is an unclean shutdown, the broker will be removed from ISR and ELR. 
   
   - Otherwise, the broker will be removed from ISR. An election can happen if 
the broker is the leader. Also, the broker can be added to ELR when possible.
   
   2. I included the integration tests for ELR in this PR.


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