Closes linkedin#70. Fix help parsing in Client to avoid triggering wh…

…en -hadoop_binary is the first option.
xkrogen · Mar 1, 2019 · 4807832 · 4807832
1 parent 9485c19
commit 4807832
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/dynamometer-infra/src/main/java/com/linkedin/dynamometer/Client.java b/dynamometer-infra/src/main/java/com/linkedin/dynamometer/Client.java
@@ -335,12 +335,12 @@ private void printUsage() {
    */
   public boolean init(String[] args) throws ParseException, IOException {
 
-    CommandLineParser parser = new GnuParser();
-    if (parser.parse(new Options().addOption("h", "help", false, "Shows this message."), args, true).hasOption("h")) {
+    if (Arrays.stream(args).anyMatch(arg -> arg.equalsIgnoreCase("-h") || arg.equals("--help"))) {
       printUsage();
       return false;
     }
 
+    CommandLineParser parser = new GnuParser();
     CommandLine cliParser = parser.parse(opts, args);
 
     yarnClient = YarnClient.createYarnClient();