[ https://issues.apache.org/jira/browse/ZOOKEEPER-2?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Patrick Hunt updated ZOOKEEPER-2: --------------------------------- Status: Open (was: Patch Available) This patch fails "ant test", specifically I see alot of the following in the log: Exception in thread "QuorumPeer" java.lang.NullPointerException at org.apache.zookeeper.server.quorum.QuorumPeer.run(QuorumPeer.java:397) (I ran the tests 3 times, failed each time in the same way, mainline test passes fine) This is ubuntu machine with java 1.6.0_06 (single cpu/core) > Synchronization issues in QuorumPeer and FastLeader election > ------------------------------------------------------------ > > Key: ZOOKEEPER-2 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2 > Project: Zookeeper > Issue Type: Bug > Components: leaderElection > Reporter: Benjamin Reed > Assignee: Flavio Paiva Junqueira > Attachments: patch-le-polish, patch-le-polish > > > There are a couple of cases of member variables that need to be marked > volatile or surrounded in a synchronization block. A couple of examples are: > * QuorumPeer state should be synchronous > * currentVote in QuorumPeer is marked volatile, but when it's members are > often accessed individually as if they were in an atomic unit. Such code > should be changed to get a reference to the currentVote and they access > members through that reference. > * It looks like logicalClock in FastLeaderElection should be volatile. It > should either be fixed or commented to explain why it doesn't need to be. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.