[8u] RFR: 8249677: Regression after JDK-8237117: Better ForkJoinPool behavior

Anton Kozlov akozlov at azul.com
Wed Jul 22 05:15:39 UTC 2020



On 21.07.2020 23:27, Andrew Hughes wrote:
> As regards the second issue which also applies to 11u, I suggest filing
> a separate bug for that to avoid confusion.

Thanks, makes sense. It is JDK-8249846: Change of behavior after JDK-8237117

>>   http://cr.openjdk.java.net/~akozlov/8249677/webrev.01/
> 
> I'm happy for this to go into 8u-dev. Please flag the bug
> jdk8u-fix-request and I'll approve.

I have to disable the second test case to avoid committing failing test.
Sorry for my rush, disabled test case is 

http://cr.openjdk.java.net/~akozlov/8249677/webrev.02/

The difference between webrev.01 and .02 is below.
I'll flag the the bug immediately after review of this.

Thanks,
Anton

diff -r dc7b3f5b37dc test/java/util/concurrent/forkjoin/AccessControlContext.java
--- a/test/java/util/concurrent/forkjoin/AccessControlContext.java	Mon Jul 20 11:24:32 2020 +0300
+++ b/test/java/util/concurrent/forkjoin/AccessControlContext.java	Wed Jul 22 08:12:07 2020 +0300
@@ -23,7 +23,7 @@
 
 /*
  * @test
- * @bug 8249677
+ * @bug 8249677 8249846
  * @summary AccessControlContext should not be dropped in ForkJoinThread
  * @run main/othervm/policy=AccessControlContext.policy/timeout=20 AccessControlContext inherit
  * @run main/othervm/policy=AccessControlContext.policy/timeout=20 AccessControlContext default
@@ -90,7 +90,10 @@
             testInherit();
             break;
         case "default":
-            testDefault();
+            // Case fails because of "JDK-8249846: Change of behavior after
+            // JDK-8237117: Better ForkJoinPool behavior".
+            System.out.println("Disabled: known to fail since 8u262 (see JDK-8249846)");
+            //testDefault();
             break;
         }
 






More information about the jdk8u-dev mailing list