You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by re...@apache.org on 2021/01/12 00:28:25 UTC

[cxf] branch master updated: Temporary workaround to open java.util.concurrent, caused by JEP-396 integration into JDK-16+

This is an automated email from the ASF dual-hosted git repository.

reta pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cxf.git


The following commit(s) were added to refs/heads/master by this push:
     new 9f939ee  Temporary workaround to open java.util.concurrent, caused by JEP-396 integration into JDK-16+
9f939ee is described below

commit 9f939eefbd41ee5c0e88b41c2e177fdbc5cbdd26
Author: reta <dr...@gmail.com>
AuthorDate: Mon Jan 11 19:27:56 2021 -0500

    Temporary workaround to open java.util.concurrent, caused by JEP-396 integration into JDK-16+
---
 core/src/main/java/org/apache/cxf/workqueue/AutomaticWorkQueueImpl.java | 1 +
 parent/pom.xml                                                          | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/core/src/main/java/org/apache/cxf/workqueue/AutomaticWorkQueueImpl.java b/core/src/main/java/org/apache/cxf/workqueue/AutomaticWorkQueueImpl.java
index 5f8a0ce..e59e7ff 100644
--- a/core/src/main/java/org/apache/cxf/workqueue/AutomaticWorkQueueImpl.java
+++ b/core/src/main/java/org/apache/cxf/workqueue/AutomaticWorkQueueImpl.java
@@ -436,6 +436,7 @@ public class AutomaticWorkQueueImpl implements AutomaticWorkQueue {
                     int sz2 = this.getActiveCount();
 
                     if ((sz + sz2) > ps) {
+                        // Needs --add-opens java.base/java.util.concurrent=ALL-UNNAMED for JDK16+
                         ReflectionUtil.setAccessible(addWorkerMethod).invoke(executor, addWorkerArgs);
                     }
                 } catch (Exception exc) {
diff --git a/parent/pom.xml b/parent/pom.xml
index 5aa54cb..30c5760 100644
--- a/parent/pom.xml
+++ b/parent/pom.xml
@@ -2383,7 +2383,7 @@
             </activation>
             <properties>
                 <!-- EasyMock stills uses JDK Internal APIs: see please https://bugs.openjdk.java.net/browse/JDK-8255363 -->
-                <cxf.surefire.fork.vmargs>-ea --add-opens java.base/java.lang=ALL-UNNAMED --add-exports java.xml/com.sun.org.apache.xerces.internal.impl.xs=ALL-UNNAMED --add-exports java.xml/com.sun.org.apache.xerces.internal.impl.xs.util=ALL-UNNAMED</cxf.surefire.fork.vmargs>
+                <cxf.surefire.fork.vmargs>-ea --add-opens java.base/java.lang=ALL-UNNAMED --add-exports java.xml/com.sun.org.apache.xerces.internal.impl.xs=ALL-UNNAMED --add-exports java.xml/com.sun.org.apache.xerces.internal.impl.xs.util=ALL-UNNAMED --add-opens java.base/java.util.concurrent=ALL-UNNAMED</cxf.surefire.fork.vmargs>
             </properties>
         </profile>
     </profiles>