[ZOOKEEPER-1227] Zookeeper logs is showing -1 as min/max session timeout if there is no sessiontimeout value configured

• Type: Sub-task
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.3.3
• Fix Version/s: 3.5.0
• Component/s: server
• Assignee: Rakesh Radhakrishnan
• Reporter: Rakesh Radhakrishnan
• Created: 14/Oct/11 14:04
• Updated: 25/Mar/14 21:15
• Resolved: 25/Mar/14 21:15

Description

When starting the ZooKeeper without configuring 'minimumSessionTimeOut' and 'maximumSessionTimeOut'.

I'm seeing the '-1' as the lower and the upper bound, instead it should give the default values : tickTime*2 and tickTime*20

2011-10-14 13:07:18,761 - INFO  [main:QuorumPeerConfig@92] - Reading configuration from: /home/amith/CI/source/install/zookeeper/zookeeper1/bin/../conf/zoo.cfg

2011-10-14 13:07:19,118 - INFO  [main:QuorumPeer@834] - tickTime set to 2000
2011-10-14 13:07:19,119 - INFO  [main:QuorumPeer@845] - minSessionTimeout set to -1
2011-10-14 13:07:19,119 - INFO  [main:QuorumPeer@856] - maxSessionTimeout set to -1

Suggestion
Move the defaulting logic to the QuorumPeerConfig instead of doing in the QuorumPeer

Issue Links

• incorporates: ZOOKEEPER-1213 ZooKeeper server startup fails if configured only with the 'minSessionTimeout' and not 'maxSessionTimeout'