From 075ec3027b19a34c49b88168e19d4c339afb898d Mon Sep 17 00:00:00 2001
From: Nihal Jain <nihaljain@apache.org>
Date: Tue, 13 Aug 2024 20:19:50 +0530
Subject: [PATCH] HBASE-28773 Failed to start HBase MiniCluster when running UT
 with JDK 17 in hbase-hbck2 (#145)

- As a generic fix we have sync'ed jvm flags from bin/hbase in main repo instead of having only specific ones needed by the code.

Signed-off-by: Istvan Toth <stoty@apache.org>
Reviewed-by: Peng Lu <lupeng_nwpu@qq.com>
---
 pom.xml | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/pom.xml b/pom.xml
index 3ec62435b0..caa032a951 100644
--- a/pom.xml
+++ b/pom.xml
@@ -146,9 +146,21 @@
 
     <jacocoArgLine/>
     <hbase-surefire.argLine>-Djava.security.manager=allow</hbase-surefire.argLine>
-    <!-- Currently, all of these options are known to be required by the test cases -->
-    <hbase-surefire.jdk11.flags>--add-opens java.base/java.lang=ALL-UNNAMED
-      --add-opens java.base/java.lang.reflect=ALL-UNNAMED</hbase-surefire.jdk11.flags>
+    <!-- Keep in sync with jvm flags in bin/hbase in main repo; Copied from there! -->
+    <hbase-surefire.jdk11.flags>-Dorg.apache.hbase.thirdparty.io.netty.tryReflectionSetAccessible=true
+      --add-modules jdk.unsupported
+      --add-opens java.base/java.io=ALL-UNNAMED
+      --add-opens java.base/java.nio=ALL-UNNAMED
+      --add-opens java.base/sun.nio.ch=ALL-UNNAMED
+      --add-opens java.base/java.lang=ALL-UNNAMED
+      --add-opens java.base/jdk.internal.ref=ALL-UNNAMED
+      --add-opens java.base/java.lang.reflect=ALL-UNNAMED
+      --add-opens java.base/java.util=ALL-UNNAMED
+      --add-opens java.base/java.util.concurrent=ALL-UNNAMED
+      --add-exports java.base/jdk.internal.misc=ALL-UNNAMED
+      --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>
     <hbase-surefire.jdk17.flags/>
   </properties>
   <dependencyManagement>