----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19522/#review38492 -----------------------------------------------------------
./src/java/test/org/apache/zookeeper/server/ZooKeeperServerMainTest.java <https://reviews.apache.org/r/19522/#comment70699> nit: spaces: ``` if (configs != null) { ``` ./src/java/test/org/apache/zookeeper/server/ZooKeeperServerMainTest.java <https://reviews.apache.org/r/19522/#comment70696> nit: just String hostport, no need to make it all uppercase like a constant (sorry i missed this before). - Raul Gutierrez Segales On March 25, 2014, 6:01 p.m., Rakesh R wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19522/ > ----------------------------------------------------------- > > (Updated March 25, 2014, 6:01 p.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 1581425 > ./src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java 1581425 > ./src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java > 1581425 > ./src/java/test/org/apache/zookeeper/server/ZooKeeperServerMainTest.java > 1581425 > ./src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java > 1581425 > ./src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerTestBase.java > 1581425 > > Diff: https://reviews.apache.org/r/19522/diff/ > > > Testing > ------- > > Tests covering standalone & quorum cases. > > > Thanks, > > Rakesh R > >
