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 2019/01/15 21:41:32 UTC

[GitHub] jzgithub1 commented on a change in pull request #897: Fixes #810 - Replace MiniAccumulo LogWriter's with ProcessBuilder.red…

jzgithub1 commented on a change in pull request #897: Fixes #810 - Replace MiniAccumulo LogWriter's with ProcessBuilder.red…
URL: https://github.com/apache/accumulo/pull/897#discussion_r248073594
 
 

 ##########
 File path: test/src/main/java/org/apache/accumulo/test/AuditMessageIT.java
 ##########
 @@ -112,10 +111,6 @@ private static long findAuditMessage(ArrayList<String> input, String pattern) {
       throw new IOException("Interrupted waiting for data to be flushed to output streams");
     }
 
-    for (MiniAccumuloClusterImpl.LogWriter lw : getCluster().getLogWriters()) {
-      lw.flush();
 
 Review comment:
   I ran 'mvn test' and there were no failures but I will investigate the point you have made.  Thank you.

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