I am wondering if the code will be more readable by removing this method and 
putting these three lines in `elect()` directly. The method is used only once 
and it is very short. And the two additional lines used to update the in-memory 
controllerContext seems more inline with the update of `activeControllerId` in 
`elect` than with the name of `tryRegisterController()`.



[ Full content available at: https://github.com/apache/kafka/pull/5101 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to