On 1/24/19 8:14 AM, Daniel Fuchs wrote:
Hi, Please find below another test fix for: 8195716: BootstrapLoggerTest : Executor still alive https://bugs.openjdk.java.net/browse/JDK-8195716 webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8195716/webrev.00/
Looks okay. This gets quite complicated to verify the executor gets GC'ed. It may be helpful to add a comment to describe the situation when a new one may be created. The test can convert to use StackWalker rather than Thread::getAllStackTraces and it can then filter with Class instance. Mandy
