-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/240/
-----------------------------------------------------------

(Updated 2011-01-10 04:30:27.786025)


Review request for zookeeper and fpj.


Changes
-------

Made changes per suggestions. 


Summary
-------

QuorumCnxManager performed blocking socket IO at a few places. As a result, QCM 
on a peer could block forever which would prevent other peers from connecting 
to the blocked peer.
If the peer happens to be the leader, then it will block new peers from 
becoming a follower.

I have made changes as per ZOOKEEPER-932


This addresses bug ZOOKEEPER-932.
    https://issues.apache.org/jira/browse/ZOOKEEPER-932


Diffs (updated)
-----

  trunk/src/java/main/org/apache/zookeeper/server/quorum/QuorumCnxManager.java 
1057153 
  trunk/src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java 
1057153 
  trunk/src/java/test/org/apache/zookeeper/test/CnxManagerTest.java 1057153 

Diff: https://reviews.apache.org/r/240/diff


Testing
-------

- ant test-core-java
- systest
- basic hand testing
- rebooted follower/leader several times


Thanks,

Vishal

Reply via email to