[CURATOR-10] NPE from JMX infrastructure in curator-test

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.0-incubating
• Fix Version/s: 2.1.0-incubating
• Component/s: General
• Assignee: Jordan Zimmerman
• Reporter: Shevek
• Created: 02/Apr/13 19:40
• Updated: 10/Jun/13 22:38
• Resolved: 10/Jun/13 22:37

Description

QuorumPeer.java contains this:

finally

{ MBeanRegistry.getInstance().unregisterAll(); }

Since MBeanRegistry is static, this unregisters all MBeans from ALL QuorumPeers in a TestingCluster.

This causes:

[ERROR] NIOServerCnxnFactory - Thread Thread[QuorumPeer[myid=1]/0:0:0:0:0:0:0:0:41733,5,main] died <java.lang.AssertionError>java.lang.AssertionError
at org.apache.zookeeper.jmx.MBeanRegistry.register(MBeanRegistry.java:89)
at org.apache.zookeeper.server.quorum.FastLeaderElection.lookForLeader(FastLeaderElection.java:771)
at org.apache.zookeeper.server.quorum.QuorumPeer.run(QuorumPeer.java:912)

and similar when connecting a new client to a server after killing another in a test suite.

Looking for a workaround, but bringing this to your attention early.

Note: Code against ZK 3.5.x library, although that unregisterAll() call seems to predate that by a long way, so this is presumably valid throughout?