[SUREFIRE-2019] ThreadedStreamConsumer - use Thread.join() instead of CountDownLatch.await()

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.22.3, 3.0.0-M6
• Component/s: Maven Failsafe Plugin, Maven Surefire Plugin, process forking
• Assignee: Tibor Digana
• Reporter: Tibor Digana
• Created: 21/Feb/22 08:37
• Updated: 21/Feb/22 08:47
• Resolved: 21/Feb/22 08:44

Description

Currently, ThreadedStreamConsumer uses CountDownLatch.await() which is not 100% reliable. The thread may die if it is in the waiting state during shutdown period. Let's use Thread.join() in the close().
Attention: do NOT use Thread.join() in the method handleEvent() - too slow, and therefore we use AtomicBoolean as a treatment.