You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by ad...@apache.org on 2017/10/10 18:05:08 UTC

kudu git commit: java client: remove JDK8 library calls from MiniKuduCluster

Repository: kudu
Updated Branches:
  refs/heads/master 8455f0374 -> c929a0605


java client: remove JDK8 library calls from MiniKuduCluster

Although we allow JDK8 usage, it's just not important enough in this case to
warrant the difference.

Change-Id: I35e08dd6e02adb8dabe45852ca892f56b9135d33
Reviewed-on: http://gerrit.cloudera.org:8080/8248
Reviewed-by: Jean-Daniel Cryans <jd...@apache.org>
Tested-by: Adar Dembo <ad...@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/c929a060
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/c929a060
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/c929a060

Branch: refs/heads/master
Commit: c929a06052d6a77d1abba52cec57214f6225fb0f
Parents: 8455f03
Author: Adar Dembo <ad...@cloudera.com>
Authored: Tue Oct 10 10:11:30 2017 -0700
Committer: Adar Dembo <ad...@cloudera.com>
Committed: Tue Oct 10 18:04:50 2017 +0000

----------------------------------------------------------------------
 .../src/test/java/org/apache/kudu/client/MiniKuduCluster.java | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/c929a060/java/kudu-client/src/test/java/org/apache/kudu/client/MiniKuduCluster.java
----------------------------------------------------------------------
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/MiniKuduCluster.java b/java/kudu-client/src/test/java/org/apache/kudu/client/MiniKuduCluster.java
index 3546d27..962bf46 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/MiniKuduCluster.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/MiniKuduCluster.java
@@ -23,7 +23,6 @@ import java.io.InputStreamReader;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
-import java.util.concurrent.TimeUnit;
 
 import com.google.common.base.Joiner;
 import com.google.common.base.Preconditions;
@@ -389,10 +388,10 @@ public class MiniKuduCluster implements AutoCloseable {
     }
     if (miniCluster != null) {
       try {
-        miniCluster.waitFor(10, TimeUnit.SECONDS);
+        miniCluster.waitFor();
       } catch (InterruptedException e) {
-        LOG.warn("Minicluster process did not exit after 10s, destroying");
-        miniCluster.destroyForcibly();
+        LOG.warn("Minicluster process did not exit, destroying");
+        miniCluster.destroy();
       }
     }
   }