[GitHub] [accumulo] Manno15 commented on a change in pull request #1728: Closes #1718. Fail if no Accumulo Instance on Standalone setup

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

Manno15 commented on a change in pull request #1728:
URL: https://github.com/apache/accumulo/pull/1728#discussion_r501914786



##########
File path: test/src/main/java/org/apache/accumulo/harness/AccumuloClusterHarness.java
##########
@@ -217,6 +219,17 @@ public void setupCluster() throws Exception {
     }
   }
 
+  public void checkConnection() {
+    try (AccumuloClient client = Accumulo.newClient().from(getClientProps()).build()) {
+      // Use prefix so cleanupTables deletes it
+      String tablePrefix = this.getClass().getSimpleName() + "_";
+      client.tableOperations().create(tablePrefix);

Review comment:
       Yeah, list is a good one that I forgot to try. I liked the instance operations for the same reasoning. I can push your suggestion tomorrow if thats more preferred.




----------------------------------------------------------------
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.

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