(amoro) branch master updated: [Improvement]: Disable jmxremote when jmxremote.port is not set. (#3166)

[czy006]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 21788902e [Improvement]: Disable jmxremote when jmxremote.port is not 
set. (#3166)
21788902e is described below

commit 21788902e8ae41701968b781e4259b9013462238
Author: baiyangtx <xiangneb...@163.com>
AuthorDate: Wed Sep 4 14:23:13 2024 +0800

    [Improvement]: Disable jmxremote when jmxremote.port is not set. (#3166)
    
    Disable jmxremote when jmxremote.port is not set.
---
 dist/src/main/amoro-bin/bin/ams.sh | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/dist/src/main/amoro-bin/bin/ams.sh 
b/dist/src/main/amoro-bin/bin/ams.sh
index 22115875d..8fa838b89 100644
--- a/dist/src/main/amoro-bin/bin/ams.sh
+++ b/dist/src/main/amoro-bin/bin/ams.sh
@@ -28,9 +28,6 @@ JAVA_OPTS="-server -XX:+UseG1GC -XX:MaxGCPauseMillis=200 \
 -Xloggc:$AMORO_LOG_DIR/gc.log -XX:+PrintGCDateStamps 
-XX:+IgnoreUnrecognizedVMOptions -XX:+UseGCLogFileRotation 
-XX:NumberOfGCLogFiles=10 -XX:GCLogFileSize=10M \
 -Xms${JVM_XMS_CONFIG}m -Xmx${JVM_XMX_CONFIG}m \
 -verbose:gc -XX:+PrintGCDetails \
--Dcom.sun.management.jmxremote \
--Dcom.sun.management.jmxremote.ssl=false \
--Dcom.sun.management.jmxremote.authenticate=false \
 --add-opens=java.base/java.lang=ALL-UNNAMED \
 --add-opens=java.base/java.lang.invoke=ALL-UNNAMED \
 --add-opens=java.base/java.lang.reflect=ALL-UNNAMED \
@@ -47,7 +44,11 @@ JAVA_OPTS="-server -XX:+UseG1GC -XX:MaxGCPauseMillis=200 \
 "
 
 if [ -n "$JMX_REMOTE_PORT_CONFIG" ];then
-        JAVA_OPTS="${JAVA_OPTS} 
-Dcom.sun.management.jmxremote.port=${JMX_REMOTE_PORT_CONFIG} "
+  JAVA_OPTS="${JAVA_OPTS} 
-Dcom.sun.management.jmxremote.port=${JMX_REMOTE_PORT_CONFIG} \
+  -Dcom.sun.management.jmxremote \
+  -Dcom.sun.management.jmxremote.ssl=false \
+  -Dcom.sun.management.jmxremote.authenticate=false \
+  "
 fi
 
 if [ ! -z "$JVM_EXTRA_CONFIG" ];then