> On March 23, 2014, 9:23 p.m., michim wrote: > > ./src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java, > > line 283 > > <https://reviews.apache.org/r/19522/diff/2/?file=531098#file531098line283> > > > > Would it be possible to avoid duplicating this logic in > > QuorumPeerConfig.java and ZooKeeperServer.java? > > > > nit: wrap these lines at 80 characters.
Thanks Michi for the review. Yes true, but I've seen ZooKeeperServerBean has exposed setMinSessionTimeout() call, in that case ZooKeeperServer will use min/max session timeout conditions. - Rakesh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19522/#review38257 ----------------------------------------------------------- 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 > >
