[ZOOKEEPER-217] handle errors when parsing config file, throw illegalargumentexception rather than exit()

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.0.0
• Fix Version/s: 3.0.1, 3.1.0
• Component/s: server
• Hadoop Flags: Reviewed
• Assignee: Mahadev Konar
• Reporter: Flavio Paiva Junqueira
• Created: 05/Nov/08 17:50
• Updated: 02/May/13 02:29
• Resolved: 18/Nov/08 17:24

Description

Discussing 209 with Ben today, we thought that it would be better to have the parse method of QuorumPeerConfig returning a boolean that indicates whether the configuration is good or not, and let the caller decide whether to exit or not. Currently we execute a System.exit() on QuorumPeerConfig.parse when we have a critical configuration error.

Issue Links

• is depended upon by: ZOOKEEPER-209 nullpointerexception if election port is not specified.