[ZOOKEEPER-2470] ServerConfig#parse(String[]) ignores tickTime

• Type: Bug
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 3.4.7, 3.5.1
• Fix Version/s: 3.4.10, 3.5.3, 3.6.0
• Component/s: server
• Assignee: Edward Ribeiro
• Reporter: Alexander Shraer
• Created: 08/Jul/16 04:59
• Updated: 31/Mar/17 09:01
• Resolved: 22/Dec/16 03:12

Description

Based on bug report from ykgarfield:

ServerConfig#parse(String[]) method has the following code:

public void parse(String[] args) {
...
if (args.length == 3)

{ tickTime = Integer.parseInt(args[2]); }

if (args.length == 4)

{ maxClientCnxns = Integer.parseInt(args[3]); }

}
```

So if args.length == 4 tickTime isn't parsed.

Issue Links

• relates to: ZOOKEEPER-2656 Fix ServerConfigTest#testValidArguments test case failures
• links to: GitHub Pull Request #125