KAFKA-17162: join() started thread in DefaultTaskManagerTest (#18570)

Reviewers: Lucas Brutschy <[email protected]>
apache · Jan 28, 2025 · dc396f4 · dc396f4
1 parent e89b30d
commit dc396f4
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/.../test/java/org/apache/kafka/streams/processor/internals/tasks/DefaultTaskManagerTest.java b/.../test/java/org/apache/kafka/streams/processor/internals/tasks/DefaultTaskManagerTest.java
@@ -137,6 +137,7 @@ public void shouldBlockOnAwait() throws InterruptedException {
         assertFalse(awaitingRunnable.awaitDone.await(100, TimeUnit.MILLISECONDS));
 
         awaitingRunnable.shutdown();
+        awaitingThread.join();
     }
 
     @Test
@@ -151,6 +152,7 @@ public void shouldReturnFromAwaitOnInterruption() throws InterruptedException {
         assertTrue(awaitingRunnable.awaitDone.await(VERIFICATION_TIMEOUT, TimeUnit.MILLISECONDS));
 
         awaitingRunnable.shutdown();
+        awaitingThread.join();
     }
 
     @Test
@@ -165,6 +167,7 @@ public void shouldReturnFromAwaitOnSignalProcessableTasks() throws InterruptedEx
         assertTrue(awaitingRunnable.awaitDone.await(VERIFICATION_TIMEOUT, TimeUnit.MILLISECONDS));
 
         awaitingRunnable.shutdown();
+        awaitingThread.join();
     }
 
     @Test
@@ -184,6 +187,7 @@ public void shouldReturnFromAwaitOnUnassignment() throws InterruptedException {
         assertTrue(awaitingRunnable.awaitDone.await(VERIFICATION_TIMEOUT, TimeUnit.MILLISECONDS));
 
         awaitingRunnable.shutdown();
+        awaitingThread.join();
     }
 
     @Test
@@ -198,6 +202,7 @@ public void shouldReturnFromAwaitOnAdding() throws InterruptedException {
         assertTrue(awaitingRunnable.awaitDone.await(VERIFICATION_TIMEOUT, TimeUnit.MILLISECONDS));
 
         awaitingRunnable.shutdown();
+        awaitingThread.join();
     }
 
     @Test
@@ -214,6 +219,7 @@ public void shouldReturnFromAwaitOnUnlocking() throws InterruptedException {
         assertTrue(awaitingRunnable.awaitDone.await(VERIFICATION_TIMEOUT, TimeUnit.MILLISECONDS));
 
         awaitingRunnable.shutdown();
+        awaitingThread.join();
     }
 
     @Test