KUDU-1669. Java ITClient test can orphan processes

This removes the interruption of threads, which is the reason why we were orphaning processes (in the ChaosThread). Change-Id: I0085323ba9ea544488c6ce896d8627f24ea69f4b Reviewed-on: http://gerrit.cloudera.org:8080/4598 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins
lsdlinshunda · Oct 5, 2016 · 3c33847 · 3c33847
1 parent 98130e3
commit 3c33847
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/ITClient.java b/java/kudu-client/src/test/java/org/apache/kudu/client/ITClient.java
@@ -116,8 +116,8 @@ public void test() throws Exception {
     KEEP_RUNNING_LATCH.countDown();
 
     for (Thread thread : threads) {
-      thread.interrupt();
-      thread.join();
+      // Give plenty of time for threads to stop.
+      thread.join(DEFAULT_SLEEP);
     }
 
     AsyncKuduScanner scannerBuilder = localAsyncClient.newScannerBuilder(table).build();
@@ -166,7 +166,6 @@ public void run() {
           }
           KEEP_RUNNING_LATCH.await(5, TimeUnit.SECONDS);
         } catch (InterruptedException e) {
-          e.printStackTrace();
           return;
         }