zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From lvfangmin <...@git.apache.org>
Subject [GitHub] zookeeper issue #628: ZOOKEEPER-3140: Allow Followers to host Observers
Date Thu, 06 Dec 2018 18:50:19 GMT
Github user lvfangmin commented on the issue:

    https://github.com/apache/zookeeper/pull/628
  
    Thanks @enixon for doing the update and rebase, went through this again, looks legit to
me. I also compared with internal version and made sure this has included all the improvement
and bug fixes.
    
    Maybe consider to add the om_proposal_process_time_ms and om_commit_process_time_ms metric,
but this is not a must have.
    
    +1 this should be ready to go.



---

Mime
View raw message