zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From anmolnar <...@git.apache.org>
Subject [GitHub] zookeeper issue #546: ZOOKEEPER-3066 Expose on JMX of Followers the id of th...
Date Mon, 25 Jun 2018 12:07:14 GMT
Github user anmolnar commented on the issue:

    https://github.com/apache/zookeeper/pull/546
  
    @eolivelli Sorry, my fault. I missed that your method is used from RemotePeerBeen too
with the remote peer id. Which is fine.
    
    Regarding unit testing.
    - validate currentVote == null case
    - validate id == getId() and id != getId() cases
    Not sure if you have to use Moquito, because `QuorumPeer` has default constructor.


---

Mime
View raw message