HBASE-25611 ExportSnapshot chmod flag uses value as decimal (#3003)

Signed-off-by: Wellington Chevreuil <wchevreuil@apache.org>

Author: petersomogyi

hbase-common/src/main/java/org/apache/hadoop/hbase/util/AbstractHBaseTool.java

@@ -234,16 +234,24 @@ protected void addOptWithArg(String shortOpt, String longOpt, String description
   }
 
   public int getOptionAsInt(CommandLine cmd, String opt, int defaultValue) {
+    return getOptionAsInt(cmd, opt, defaultValue, 10);
+  }
+
+  public int getOptionAsInt(CommandLine cmd, String opt, int defaultValue, int radix) {
     if (cmd.hasOption(opt)) {
-      return Integer.parseInt(cmd.getOptionValue(opt));
+      return Integer.parseInt(cmd.getOptionValue(opt), radix);
     } else {
       return defaultValue;
     }
   }
 
   public long getOptionAsLong(CommandLine cmd, String opt, int defaultValue) {
+    return getOptionAsLong(cmd, opt, defaultValue, 10);
+  }
+
+  public long getOptionAsLong(CommandLine cmd, String opt, int defaultValue, int radix) {
     if (cmd.hasOption(opt)) {
-      return Long.parseLong(cmd.getOptionValue(opt));
+      return Long.parseLong(cmd.getOptionValue(opt), radix);
     } else {
       return defaultValue;
     }

hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/snapshot/ExportSnapshot.java

@@ -927,7 +927,7 @@ protected void processOptions(CommandLine cmd) {
     mappers = getOptionAsInt(cmd, Options.MAPPERS.getLongOpt(), mappers);
     filesUser = cmd.getOptionValue(Options.CHUSER.getLongOpt(), filesUser);
     filesGroup = cmd.getOptionValue(Options.CHGROUP.getLongOpt(), filesGroup);
-    filesMode = getOptionAsInt(cmd, Options.CHMOD.getLongOpt(), filesMode);
+    filesMode = getOptionAsInt(cmd, Options.CHMOD.getLongOpt(), filesMode, 8);
     bandwidthMB = getOptionAsInt(cmd, Options.BANDWIDTH.getLongOpt(), bandwidthMB);
     overwrite = cmd.hasOption(Options.OVERWRITE.getLongOpt());
     // And verifyChecksum and verifyTarget with values read from old args in processOldArgs(...).