GitHub user junrao opened a pull request: https://github.com/apache/kafka/pull/4351
kafka-6320: move ZK metrics in KafkaHealthCheck to ZookeeperClient * Moved metrics in KafkaHealthCheck to ZookeeperClient. * Converted remaining ZkUtils usage in KafkaServer to ZookeeperClient and removed ZkUtils from KafkaServer. * Made the re-creation of ZooKeeper during ZK session expiration with infinite retries. * Added unit tests for all new methods in KafkaZkClient. ### Committer Checklist (excluded from commit message) - [ ] Verify design and implementation - [ ] Verify test coverage and CI build status - [ ] Verify documentation (including upgrade notes) You can merge this pull request into a Git repository by running: $ git pull https://github.com/junrao/kafka kafka-6320 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/4351.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #4351 ---- commit 68937522ada25ed1efdda0f51919375d997e596e Author: Jun Rao <junrao@...> Date: 2017-12-21T22:09:17Z kafka-6320: move ZK metrics in KafkaHealthCheck to ZookeeperClient ---- ---