fix jvm options

rishabhmaurya · Oct 3, 2024 · 25bf6c6 · 25bf6c6
1 parent c04cb3e
commit 25bf6c6
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 13 deletions.
diff --git a/buildSrc/src/main/java/org/opensearch/gradle/testclusters/RunTask.java b/buildSrc/src/main/java/org/opensearch/gradle/testclusters/RunTask.java
@@ -166,10 +166,8 @@ public void beforeStart() {
             firstNode.setTransportPort(String.valueOf(transportPort));
             transportPort++;
             firstNode.setting("discovery.seed_hosts", LOCALHOST_ADDRESS_PREFIX + DEFAULT_TRANSPORT_PORT);
-
             cluster.setPreserveDataDir(preserveData);
             for (OpenSearchNode node : cluster.getNodes()) {
-
                 if (node != firstNode) {
                     node.setHttpPort(String.valueOf(httpPort));
                     httpPort++;
@@ -197,7 +195,6 @@ public void beforeStart() {
                 if (keystorePassword.length() > 0) {
                     node.keystorePassword(keystorePassword);
                 }
-                node.jvmArgs("--add-opens=java.base/java.nio=ALL-UNNAMED");
             }
         }
     }

diff --git a/distribution/src/config/jvm.options b/distribution/src/config/jvm.options
@@ -85,5 +85,4 @@ ${error.file}
 
 # HDFS ForkJoinPool.common() support by SecurityManager
 -Djava.util.concurrent.ForkJoinPool.common.threadFactory=org.opensearch.secure_sm.SecuredForkJoinWorkerThreadFactory
--Dio.netty.tryReflectionSetAccessible=true
--XX:MaxDirectMemorySize=1g
+--add-opens=java.base/java.nio=org.apache.arrow.memory.core,ALL-UNNAMED
diff --git a/server/build.gradle b/server/build.gradle
@@ -420,14 +420,6 @@ tasks.test {
         jvmArgs += ["--add-opens", "java.base/java.nio.file=ALL-UNNAMED"]
     }
 }
-
-tasks.test {
-  //if (BuildParams.runtimeJavaVersion > JavaVersion.VERSION_1_8) {
-    jvmArgs += ["--add-opens", "java.base/java.nio.channels=ALL-UNNAMED"]
-    jvmArgs += ["--add-opens", "java.base/java.net=ALL-UNNAMED"]
-  //}
-}
-
 tasks.named("sourcesJar").configure {
   // Ignore duplicates for protobuf generated code (main and generatedSources).
   filesMatching("**/proto/*") {