----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19522/#review38257 -----------------------------------------------------------
./src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java <https://reviews.apache.org/r/19522/#comment70310> Would it be possible to avoid duplicating this logic in QuorumPeerConfig.java and ZooKeeperServer.java? nit: wrap these lines at 80 characters. ./src/java/test/org/apache/zookeeper/server/ZooKeeperServerMainTest.java <https://reviews.apache.org/r/19522/#comment70311> nit: wrap this line at 80 characters. ./src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerTestBase.java <https://reviews.apache.org/r/19522/#comment70312> nit: 80 characters - michim On March 21, 2014, 7:58 a.m., Rakesh R wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19522/ > ----------------------------------------------------------- > > (Updated March 21, 2014, 7:58 a.m.) > > > Review request for zookeeper, fpj, michim, Patrick Hunt, and Raul Gutierrez > Segales. > > > Bugs: ZOOKEEPER-1263 > https://issues.apache.org/jira/browse/ZOOKEEPER-1263 > > > Repository: zookeeper > > > Description > ------- > > Fix handling of min/max session timeout value initialization. Addresses > foollowing cases:- > 1) Server startup fails if configured only with the 'minSessionTimeout' and > not 'maxSessionTimeout' > 2) Log is showing -1 as min/max session timeout if there is no sessiontimeout > value configured > > > Diffs > ----- > > ./src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java 1579879 > ./src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java 1579879 > ./src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java > 1579879 > ./src/java/test/org/apache/zookeeper/server/ZooKeeperServerMainTest.java > 1579879 > ./src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java > 1579879 > ./src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerTestBase.java > 1579879 > > Diff: https://reviews.apache.org/r/19522/diff/ > > > Testing > ------- > > Tests covering standalone & quorum cases. > > > Thanks, > > Rakesh R > >
