[GitHub] [hbase-operator-tools] clarax commented on a diff in pull request #105: HBASE-24587 hbck2 command should accept one or more files containing a list of region names/table names/namespaces

[GitBox <gi...@apache.org>]

clarax commented on code in PR #105:
URL: https://github.com/apache/hbase-operator-tools/pull/105#discussion_r846583703


##########
hbase-hbck2/src/main/java/org/apache/hbase/HBCK2.java:
##########
@@ -185,6 +185,37 @@ void checkFunctionSupported(ClusterConnection connection, String cmd) throws IOE
     }
   }
 
+  void setTableState(Hbck hbck, String[] args) throws IOException {
+    Options options = new Options();
+    Option inputFile = Option.builder("i").longOpt("inputFiles").build();
+    options.addOption(inputFile);
+    CommandLine commandLine = getCommandLine(args, options);
+    if (commandLine == null) {
+      return;
+    }
+    String[] argList = commandLine.getArgs();
+    if(!commandLine.hasOption(inputFile.getOpt())) {
+      System.out.println(setTableStateByArgs(hbck, argList));

Review Comment:
   This is the existing behavior. it is ok to keep it  and I don't want to break any existing automations.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@hbase.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org