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

Raul Gutierrez Segales commented on ZOOKEEPER-2239:
---------------------------------------------------

The failure is unrelated:

{code}
1 tests failed.
REGRESSION:  org.apache.zookeeper.test.ReconfigTest.testPortChange

Error Message:
expected:<test[1]> but was:<test[0]>

Stack Trace:
junit.framework.AssertionFailedError: expected:<test[1]> but was:<test[0]>
        at 
org.apache.zookeeper.test.ReconfigTest.testNormalOperation(ReconfigTest.java:150)
        at 
org.apache.zookeeper.test.ReconfigTest.testPortChange(ReconfigTest.java:598)
        at 
org.apache.zookeeper.JUnit4ZKTestRunner$LoggedInvokeMethod.evaluate(JUnit4ZKTestRunner.java:79)
{code}

So merging regardless. 

> JMX State from LocalPeerBean Incorrect
> --------------------------------------
>
>                 Key: ZOOKEEPER-2239
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2239
>             Project: ZooKeeper
>          Issue Type: Bug
>          Components: jmx
>    Affects Versions: 3.4.6, 3.5.0, 3.5.1
>         Environment: All
>            Reporter: Kevin Lee
>             Fix For: 3.4.7, 3.5.2, 3.6.0
>
>         Attachments: ZOOKEEPER-2239.patch
>
>
> The "State" property of LocalPeerBean in package 
> org.apache.zookeeper.server.quorum is returning the incorrect value.  It is 
> performing peer.getState() which is calling the getState() method on 
> java.lang.Thread instead of getting the server state from 
> org.apache.zookeeper.server.quorum.QuorumPeer.  The Javadoc within 
> LocalPeerMXBean.java states that it should be returning the server state as 
> well.  The fix is to call peer.getServerState() in the getState() method of 
> LocalPeerBean instead of peer.getState().toString().  This will return the 
> states defined in QuorumStats.Provider (unknown, leaderelection, leading, 
> following, and observing).  This issue prevents JMX monitoring of the 
> Zookeeper server state.



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

Reply via email to