[RATIS-524] JCommander args should have `required=true` when appropriate

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.4.0
• Component/s: LogService
• Assignee: Josh Elser
• Reporter: Josh Elser
• Created: 10/Apr/19 19:13
• Updated: 03/May/19 18:07
• Resolved: 03/May/19 18:07

Description

VerificationTool and the LogServiceShell both use JCommander to implement a required argument provided by the user (the metadata quorum string), but don't mark the option as required. This lets the user omit providing it which results in a NPE.