zookeeper-dev mailing list archives

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

    https://github.com/apache/zookeeper/pull/546
  
    @anmolnar 
    
    I checked the code, in my opinion the *id* inside current vote is the id of what the local
peer thinks the leader is. See for instance updateServerState, in which state is calculated
from the current vote.
    
    So my code should be correct and there is no need (or it will be an error) to check the
*state* variable.
    
    About an additional test....I don't know what will be the meaning of the test you are
suggesting, maybe I am missing something


---

Mime
View raw message