showuon commented on PR #15287:
URL: https://github.com/apache/kafka/pull/15287#issuecomment-1922641648

   @cmccabe , thanks for the comments. Please see my reply below:
   
   > running zookeeper-shell.sh does not need to be done "quickly together with 
rolling at least one broker". You must have all brokers in hybrid ZK mode 
before deleting the /controller znode. Having even a single broker in KRaft 
mode would of course be disastrous if the controller is ZK.
   
   I just confirmed both @ppatierno and you are correct:
   1. As Paolo said, when in hybrid ZK mode, we'll create 
`zk-broker-0-to-controller-quorum-channel-manager` to talk to KRaft 
controllers. And after Kraft controllers are deprovisioned, the hybrid ZK 
broker will flood with the error saying no controller and connecting to KRaft 
voters failures. That would indeed confuse users and treat it as "failure".
   2. Even in the above state with lots of error logs, I can confirm, after 
deleting the /controller znode, the hybrid ZK broker will take over, and 
everything works fine. (except the error logs in (1) still keep emitting)
   
   Thanks for helping the revert!


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