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 2018/12/12 20:40:48 UTC

[GitHub] mikewalch commented on a change in pull request #825: #820 - Make sure AccumuloClient is closed

mikewalch commented on a change in pull request #825:  #820 - Make sure AccumuloClient is closed
URL: https://github.com/apache/accumulo/pull/825#discussion_r241177284
 
 

 ##########
 File path: test/src/main/java/org/apache/accumulo/test/AuditMessageIT.java
 ##########
 @@ -207,7 +207,8 @@ public void testTableOperationsAudits() throws AccumuloException, AccumuloSecuri
 
     // Connect as Audit User and do a bunch of stuff.
     // Testing activity begins here
-    auditAccumuloClient = getCluster().getAccumuloClient(AUDIT_USER_1, new PasswordToken(PASSWORD));
+    auditAccumuloClient = getCluster().createAccumuloClient(AUDIT_USER_1,
 
 Review comment:
   I wanted to limit the size of this PR.  I won't close #820 until everything has been closed or other issues have been created for remaining work

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services