8169243: java/util/concurrent/ThreadPoolExecutor/ConfigChanges.java still fails intermittently
authordl
Mon, 28 Nov 2016 23:33:25 -0800
changeset 42318 35806bea1e90
parent 42317 debe8d32a0a4
child 42319 0193886267c3
8169243: java/util/concurrent/ThreadPoolExecutor/ConfigChanges.java still fails intermittently Reviewed-by: martin, smarks, psandoz, amlu
jdk/test/java/util/concurrent/ThreadPoolExecutor/ConfigChanges.java
--- a/jdk/test/java/util/concurrent/ThreadPoolExecutor/ConfigChanges.java	Mon Nov 28 17:32:49 2016 -0800
+++ b/jdk/test/java/util/concurrent/ThreadPoolExecutor/ConfigChanges.java	Mon Nov 28 23:33:25 2016 -0800
@@ -247,7 +247,13 @@
                tg.activeCount() > 0)
             Thread.sleep(4);
         equal(tg.activeCount(), 0);
-        check(System.nanoTime() - t0 >= tpe.getKeepAliveTime(NANOSECONDS));
+
+        // The following assertion is almost always true, but may
+        // exceptionally not be during a transition from core count
+        // too high to allowCoreThreadTimeOut.  Users will never
+        // notice, and we accept the small loss of testability.
+        //
+        // check(System.nanoTime() - t0 >= tpe.getKeepAliveTime(NANOSECONDS));
 
         //report("idle", tpe);