You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/08/19 15:12:50 UTC

[GitHub] [accumulo] DomGarguilo commented on a diff in pull request #2881: Cleanup Accumulo Admin utilities

DomGarguilo commented on code in PR #2881:
URL: https://github.com/apache/accumulo/pull/2881#discussion_r950255806


##########
server/base/src/main/java/org/apache/accumulo/server/util/Admin.java:
##########
@@ -226,49 +226,52 @@ public void execute(final String[] args) {
     JCommander cl = new JCommander(opts);
     cl.setProgramName("accumulo admin");
 
-    CheckTabletsCommand checkTabletsCommand = new CheckTabletsCommand();
-    cl.addCommand("checkTablets", checkTabletsCommand);
-
     ChangeSecretCommand changeSecretCommand = new ChangeSecretCommand();
     cl.addCommand("changeSecret", changeSecretCommand);
 
+    CheckTabletsCommand checkTabletsCommand = new CheckTabletsCommand();
+    cl.addCommand("checkTablets", checkTabletsCommand);
+
     DeleteZooInstanceCommand deleteZooInstanceOpts = new DeleteZooInstanceCommand();
     cl.addCommand("deleteZooInstance", deleteZooInstanceOpts);
 
-    RestoreZooCommand restoreZooOpts = new RestoreZooCommand();
-    cl.addCommand("restoreZoo", restoreZooOpts);
+    DumpConfigCommand dumpConfigCommand = new DumpConfigCommand();
+    cl.addCommand("dumpConfig", dumpConfigCommand);
 
     ListInstancesCommand listIntancesOpts = new ListInstancesCommand();
     cl.addCommand("listInstances", listIntancesOpts);

Review Comment:
   ```suggestion
       ListInstancesCommand listInstancesOpts = new ListInstancesCommand();
       cl.addCommand("listInstances", listInstancesOpts);
   ```
   I know you didn't make this change but thought it would be worth it to fix this typo while changes are being made to this file.



-- 
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: notifications-unsubscribe@accumulo.apache.org

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