You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@kafka.apache.org by junrao <gi...@git.apache.org> on 2017/12/21 22:44:38 UTC

[GitHub] kafka pull request #4351: kafka-6320: move ZK metrics in KafkaHealthCheck to...

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 <ju...@...>
Date:   2017-12-21T22:09:17Z

    kafka-6320: move ZK metrics in KafkaHealthCheck to ZookeeperClient

----


---