Now that `ControllerMovedException` may be handled differently from other exceptions, the logic would be cleaner if we use this exception only when we know the another broker is the controller.
Thinking about it more, IllegalStateException means something impossible has happened inside the controller state. In this case the exception can happen if controller fails to write to controller epoch znode, which is possible from controller's point of view since zookeeper service is out of controller of the controller. How about `ZooKeeperClientException(...)` and include error code in the message of the exception? [ Full content available at: https://github.com/apache/kafka/pull/5101 ] This message was relayed via gitbox.apache.org for [email protected]
