Prior to this patch, if setControllerEpochRaw fails and the error is not `NONODE`, ControllerMovedException will be thrown which will be caught and `triggerControllerMove()` will be executed.
After this patch, if setControllerEpochRaw returns a non-OK error code, we will always throw ControllerMovedException(), which will be caught in the upper layer without executing `triggerControllerMove()`. I am wondering if we should throw IllegalStateException if the error code suggests something other than controller move, so that we can still execute `triggerControllerMove()` in this scenario. [ Full content available at: https://github.com/apache/kafka/pull/5101 ] This message was relayed via gitbox.apache.org for [email protected]
