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

zhangduo pushed a commit to branch branch-2.5
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2.5 by this push:
     new d4277aa892b HBASE-28723 [JDK17] TestSecureIPC fails under JDK17 (#6072)
d4277aa892b is described below

commit d4277aa892be808e5a96d116ff477b02088138fa
Author: Duo Zhang <zhang...@apache.org>
AuthorDate: Sat Jul 13 22:26:19 2024 +0800

    HBASE-28723 [JDK17] TestSecureIPC fails under JDK17 (#6072)
    
    Signed-off-by: Yi Mei <me...@apache.org>
    (cherry picked from commit 9dee538f65d84a900724d424c71793dff46e9684)
---
 pom.xml | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/pom.xml b/pom.xml
index 9726de2b836..ba1bedb14be 100644
--- a/pom.xml
+++ b/pom.xml
@@ -719,8 +719,14 @@
       "-Djava.library.path=${hadoop.library.path};${java.library.path}"
       -Dorg.apache.hbase.thirdparty.io.netty.leakDetection.level=advanced
       
-Dio.opentelemetry.context.enableStrictContext=true</hbase-surefire.cygwin-argLine>
-    <!-- Keep these options in sync with add_jdk11_jvm_flags() in bin/hbase.
-    Currently, all of these options are known to be required by HBase, and not 
the test cases -->
+    <!--
+      Keep these jdk11/jdk17 options in sync with add_jdk11_jvm_flags() in 
bin/hbase, except the
+      below ones which are only used in test code
+      'java.base/jdk.internal.util.random' is required by the test code
+      'java.base/sun.security.x509' and 'java.base/sun.security.util' are 
required by
+      TestLdapHttpServer, see HBASE-28341
+      'java.base/java.net' is required by TestSecureIPC, see HBASE-28723
+    -->
     
<hbase-surefire.jdk11.flags>-Dorg.apache.hbase.thirdparty.io.netty.tryReflectionSetAccessible=true
       --add-modules jdk.unsupported
       --add-opens java.base/java.io=ALL-UNNAMED
@@ -735,11 +741,10 @@
       --add-exports java.security.jgss/sun.security.krb5=ALL-UNNAMED
       --add-exports java.base/sun.net.dns=ALL-UNNAMED
       --add-exports 
java.base/sun.net.util=ALL-UNNAMED</hbase-surefire.jdk11.flags>
-    <!-- java.base/jdk.internal.util.random=ALL-UNNAMED is required by the 
test code, so we do not
-    need this in bin/hbase -->
     <hbase-surefire.jdk17.flags>--add-opens 
java.base/jdk.internal.util.random=ALL-UNNAMED
       --add-opens java.base/sun.security.x509=ALL-UNNAMED
-      --add-opens 
java.base/sun.security.util=ALL-UNNAMED</hbase-surefire.jdk17.flags>
+      --add-opens java.base/sun.security.util=ALL-UNNAMED
+      --add-opens java.base/java.net=ALL-UNNAMED</hbase-surefire.jdk17.flags>
     <!-- Surefire argLine defaults to Linux, cygwin argLine is used in the 
os.windows profile -->
     <argLine>${hbase-surefire.argLine} @{jacocoArgLine}</argLine>
     <extra.enforcer.version>1.5.1</extra.enforcer.version>

Reply via email to