[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-2479?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15635085#comment-15635085
 ] 

ASF GitHub Bot commented on ZOOKEEPER-2479:
-------------------------------------------

Github user rakeshadr commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/98#discussion_r86483860
  
    --- Diff: src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java 
---
    @@ -520,6 +520,12 @@ public synchronized void setCurrentVote(Vote v){
         protected boolean quorumListenOnAllIPs = false;
     
         /**
    +     * Keeps time taken for leader election in milliseconds. Sets the 
value to
    +     * this variable only after the completion of leader election.
    +     */
    +    private long electionTimeTaken = -1;
    --- End diff --
    
    Thanks @eribeiro ,
    
    LeaderMXBean, FollowerMXBean will be available only after the quorum leader 
election and the value won't be changed until next LE. I think, adding 
'volatile' doesn't make any difference, right?
    
    code reference:
    
    
https://github.com/apache/zookeeper/blob/master/src/java/main/org/apache/zookeeper/server/quorum/Leader.java#L417
    
https://github.com/apache/zookeeper/blob/master/src/java/main/org/apache/zookeeper/server/quorum/Follower.java#L70
    
        Leader.java
            zk.registerJMX(new LeaderBean(this, zk), self.jmxLocalPeerBean);
    
        Follower.java
            fzk.registerJMX(new FollowerBean(this, zk), self.jmxLocalPeerBean);


> Add 'electionTimeTaken' value in LeaderMXBean and FollowerMXBean
> ----------------------------------------------------------------
>
>                 Key: ZOOKEEPER-2479
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2479
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: quorum
>            Reporter: Rakesh R
>            Assignee: Rakesh R
>             Fix For: 3.4.10, 3.5.3, 3.6.0
>
>         Attachments: ZOOKEEPER-2479.patch, ZOOKEEPER-2479.patch, 
> ZOOKEEPER-2479.patch, ZOOKEEPER-2479.patch, ZOOKEEPER-2479.patch, 
> ZOOKEEPER-2479.patch
>
>
> The idea of this jira is to expose {{time taken}} for the leader election via 
> jmx Leader, Follower beans.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to