This is an automated email from the ASF dual-hosted git repository. xianjin pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git
The following commit(s) were added to refs/heads/master by this push: new be6afb92d [#1959] improvement(script): Decouple scripts and configs (#1957) be6afb92d is described below commit be6afb92d4d6a72ba3f2b08b402d341f52104128 Author: maobaolong <baoloong...@tencent.com> AuthorDate: Tue Jul 30 10:37:12 2024 +0800 [#1959] improvement(script): Decouple scripts and configs (#1957) ### What changes were proposed in this pull request? There are many hard code defined ENV variables in start-xxx.sh, we need to reactor and abstract variables to let user be able to do customize config. ### Why are the changes needed? Scripts in the `bin` directory should be immutable and driven by configurations. Fix: #1959 ### Does this PR introduce _any_ user-facing change? User can define more environment variables in `conf/rss-env.sh` or by line env-var. ### How was this patch tested? ``` COORDINATOR_JAVA_OPTS=" -Daa=bb" COORDINATOR_CONF_FILE=`pwd`/conf/coordinator.conf COORDINATOR_XMX_SIZE=3g bin/start-coordinator.sh SHUFFLE_SERVER_XMX_SIZE=4g bin/start-shuffle-server.sh bin/start-dashboard.sh ``` --- README.md | 8 +++-- bin/start-coordinator.sh | 26 +++++++------- bin/start-dashboard.sh | 26 +++++++------- bin/start-shuffle-server.sh | 42 +++++++++++----------- conf/rss-env.sh | 26 +++++++++++--- deploy/kubernetes/docker/rss-env.sh | 21 +++++++++++ .../operator/examples/configuration.yaml | 10 +++++- docs/coordinator_guide.md | 6 ++-- docs/dashboard_guide.md | 4 +-- docs/server_guide.md | 6 ++-- 10 files changed, 115 insertions(+), 60 deletions(-) diff --git a/README.md b/README.md index f0a9e2ed2..3fc1b5692 100644 --- a/README.md +++ b/README.md @@ -155,7 +155,9 @@ If you have packaged tgz with hadoop jars, the env of `HADOOP_HOME` is needn't s ``` JAVA_HOME=<java_home> HADOOP_HOME=<hadoop home> - XMX_SIZE="16g" + COORDINATOR_XMX_SIZE="16g" + # You can set coordinator memory size by `XMX_SIZE` too, but it affects all components. + # XMX_SIZE="16g" ``` 3. update RSS_HOME/conf/coordinator.conf, e.g., ``` @@ -206,7 +208,9 @@ Deploy Steps: ``` JAVA_HOME=<java_home> HADOOP_HOME=<hadoop home> - XMX_SIZE="80g" + SHUFFLE_SERVER_XMX_SIZE="80g" + # You can set shuffle server memory size by `XMX_SIZE` too, but it affects all components. + # XMX_SIZE="80g" ``` 3. update RSS_HOME/conf/server.conf, e.g., ``` diff --git a/bin/start-coordinator.sh b/bin/start-coordinator.sh index 688675cd8..05bc4c104 100755 --- a/bin/start-coordinator.sh +++ b/bin/start-coordinator.sh @@ -60,15 +60,17 @@ set -u echo "class path is $CLASSPATH" -JVM_ARGS=" -server \ - -Xmx${XMX_SIZE:-8g} \ - -Xms${XMX_SIZE:-8g} \ - -XX:+UseG1GC \ +COORDINATOR_XMX_SIZE=${COORDINATOR_XMX_SIZE:-${XMX_SIZE:-"8g"}} +COORDINATOR_BASE_JVM_ARGS=${COORDINATOR_BASE_JVM_ARGS:-" -server \ + -Xmx${COORDINATOR_XMX_SIZE} \ + -Xms${COORDINATOR_XMX_SIZE} \ + -XX:+PrintCommandLineFlags"} + +DEFAULT_GC_ARGS=" -XX:+UseG1GC \ -XX:MaxGCPauseMillis=200 \ -XX:ParallelGCThreads=20 \ -XX:ConcGCThreads=5 \ - -XX:InitiatingHeapOccupancyPercent=45 \ - -XX:+PrintCommandLineFlags" + -XX:InitiatingHeapOccupancyPercent=45" GC_LOG_ARGS_LEGACY=" -XX:+PrintGC \ -XX:+PrintAdaptiveSizePolicy \ @@ -91,10 +93,10 @@ GC_LOG_ARGS_NEW=" -XX:+IgnoreUnrecognizedVMOptions \ -Xlog:gc+start=debug \ -Xlog:gc*:file=${RSS_LOG_DIR}/gc-%t.log:tags,uptime,time,level" -ARGS="" +JVM_LOG_ARGS="" if [ -f ${LOG_CONF_FILE} ]; then - ARGS="$ARGS -Dlog4j2.configurationFile=file:${LOG_CONF_FILE} -Dlog.path=${LOG_PATH}" + JVM_LOG_ARGS=" -Dlog4j2.configurationFile=file:${LOG_CONF_FILE} -Dlog.path=${LOG_PATH}" else echo "Exit with error: ${LOG_CONF_FILE} file doesn't exist." exit 1 @@ -102,13 +104,13 @@ fi version=$($RUNNER -version 2>&1 | awk -F[\".] '/version/ {print $2}') if [[ "$version" -lt "9" ]]; then - GC_ARGS=$GC_LOG_ARGS_LEGACY + COORDINATOR_JVM_GC_ARGS="${COORDINATOR_JVM_GC_ARGS:-${DEFAULT_GC_ARGS} ${GC_LOG_ARGS_LEGACY}}" else - GC_ARGS=$GC_LOG_ARGS_NEW + COORDINATOR_JVM_GC_ARGS="${COORDINATOR_JVM_GC_ARGS:-${DEFAULT_GC_ARGS} ${GC_LOG_ARGS_NEW}}" fi -UNIFFLE_COORDINATOR_JAVA_OPTS=${UNIFFLE_COORDINATOR_JAVA_OPTS:-""} -$RUNNER ${UNIFFLE_COORDINATOR_JAVA_OPTS} $ARGS $JVM_ARGS $GC_ARGS -cp $CLASSPATH $MAIN_CLASS --conf "$COORDINATOR_CONF_FILE" $@ & +COORDINATOR_JAVA_OPTS=${COORDINATOR_JAVA_OPTS:-""} +$RUNNER ${COORDINATOR_BASE_JVM_ARGS} ${COORDINATOR_JVM_GC_ARGS} ${JVM_LOG_ARGS} ${COORDINATOR_JAVA_OPTS} -cp ${CLASSPATH} ${MAIN_CLASS} --conf "${COORDINATOR_CONF_FILE}" $@ & get_pid_file_name coordinator echo $! >${RSS_PID_DIR}/${pid_file} diff --git a/bin/start-dashboard.sh b/bin/start-dashboard.sh index 757059d86..2b35cfa76 100755 --- a/bin/start-dashboard.sh +++ b/bin/start-dashboard.sh @@ -47,15 +47,17 @@ mkdir -p "${RSS_PID_DIR}" echo "class path is $CLASSPATH" -JVM_ARGS=" -server \ - -Xmx${XMX_SIZE:-8g} \ - -Xms${XMX_SIZE:-8g} \ - -XX:+UseG1GC \ +DASHBOARD_XMX_SIZE=${DASHBOARD_XMX_SIZE:-${XMX_SIZE:-"8g"}} +DASHBOARD_BASE_JVM_ARGS=${DASHBOARD_BASE_JVM_ARGS:-" -server \ + -Xmx${DASHBOARD_XMX_SIZE} \ + -Xms${DASHBOARD_XMX_SIZE} \ + -XX:+PrintCommandLineFlags"} + +DEFAULT_GC_ARGS=" -XX:+UseG1GC \ -XX:MaxGCPauseMillis=200 \ -XX:ParallelGCThreads=20 \ -XX:ConcGCThreads=5 \ - -XX:InitiatingHeapOccupancyPercent=45 \ - -XX:+PrintCommandLineFlags" + -XX:InitiatingHeapOccupancyPercent=45" GC_LOG_ARGS_LEGACY=" -XX:+PrintGC \ -XX:+PrintAdaptiveSizePolicy \ @@ -72,10 +74,10 @@ GC_LOG_ARGS_NEW=" -XX:+IgnoreUnrecognizedVMOptions \ -Xlog:gc*:file=${RSS_LOG_DIR}/dashboard-gc-%t.log:tags,uptime,time,level:filecount=5,filesize=102400" -ARGS="" +JVM_LOG_ARGS="" if [ -f ${LOG_CONF_FILE} ]; then - ARGS="$ARGS -Dlog4j2.configurationFile=file:${LOG_CONF_FILE} -Dlog.path=${LOG_PATH}" + JVM_LOG_ARGS=" -Dlog4j2.configurationFile=file:${LOG_CONF_FILE} -Dlog.path=${LOG_PATH}" else echo "Exit with error: ${LOG_CONF_FILE} file doesn't exist." exit 1 @@ -83,13 +85,13 @@ fi version=$($RUNNER -version 2>&1 | awk -F[\".] '/version/ {print $2}') if [[ "$version" -lt "9" ]]; then - GC_ARGS=$GC_LOG_ARGS_LEGACY + DASHBOARD_JVM_GC_ARGS="${DASHBOARD_JVM_GC_ARGS:-${DEFAULT_GC_ARGS} ${GC_LOG_ARGS_LEGACY}}" else - GC_ARGS=$GC_LOG_ARGS_NEW + DASHBOARD_JVM_GC_ARGS="${DASHBOARD_JVM_GC_ARGS:-${DEFAULT_GC_ARGS} ${GC_LOG_ARGS_NEW}}" fi -UNIFFLE_DASHBOARD_JAVA_OPTS=${UNIFFLE_DASHBOARD_JAVA_OPTS:-""} -$RUNNER ${UNIFFLE_DASHBOARD_JAVA_OPTS} $ARGS $JVM_ARGS $GC_ARGS -cp $CLASSPATH $MAIN_CLASS --conf "$DASHBOARD_CONF_FILE" $@ & +DASHBOARD_JAVA_OPTS=${DASHBOARD_JAVA_OPTS:-""} +$RUNNER ${DASHBOARD_BASE_JVM_ARGS} ${DASHBOARD_JVM_GC_ARGS} ${JVM_LOG_ARGS} ${DASHBOARD_JAVA_OPTS} -cp ${CLASSPATH} ${MAIN_CLASS} --conf "${DASHBOARD_CONF_FILE}" $@ & get_pid_file_name dashboard echo $! >${RSS_PID_DIR}/${pid_file} diff --git a/bin/start-shuffle-server.sh b/bin/start-shuffle-server.sh index 1f89d7d09..bc52c65ee 100755 --- a/bin/start-shuffle-server.sh +++ b/bin/start-shuffle-server.sh @@ -30,14 +30,15 @@ SHUFFLE_SERVER_CONF_FILE="${RSS_CONF_DIR}/server.conf" JAR_DIR="${RSS_HOME}/jars" LOG_CONF_FILE="${RSS_CONF_DIR}/log4j2.xml" LOG_PATH="${RSS_LOG_DIR}/shuffle_server.log" -SHUFFLE_SERVER_STORAGE_AUDIT_LOG_PATH="${RSS_LOG_DIR}/shuffle_server_storage_audit.log" -SHUFFLE_SERVER_RPC_AUDIT_LOG_PATH="${RSS_LOG_DIR}/shuffle_server_rpc_audit.log" +SHUFFLE_SERVER_STORAGE_AUDIT_LOG_PATH=${SHUFFLE_SERVER_STORAGE_AUDIT_LOG_PATH:-"${RSS_LOG_DIR}/shuffle_server_storage_audit.log"} +SHUFFLE_SERVER_RPC_AUDIT_LOG_PATH=${SHUFFLE_SERVER_RPC_AUDIT_LOG_PATH:-"${RSS_LOG_DIR}/shuffle_server_rpc_audit.log"} -if [ -z "${XMX_SIZE:-}" ]; then - echo "No env XMX_SIZE." +SHUFFLE_SERVER_XMX_SIZE=${SHUFFLE_SERVER_XMX_SIZE:-${XMX_SIZE}} +if [ -z "${SHUFFLE_SERVER_XMX_SIZE:-}" ]; then + echo "No env XMX_SIZE or SHUFFLE_SERVER_XMX_SIZE." exit 1 fi -echo "Shuffle Server JVM XMX size: ${XMX_SIZE}" +echo "Shuffle Server JVM XMX size: ${SHUFFLE_SERVER_XMX_SIZE}" if [ -n "${RSS_IP:-}" ]; then echo "Shuffle Server RSS_IP: ${RSS_IP}" fi @@ -92,22 +93,23 @@ if [ -n "${OOM_DUMP_PATH:-}" ]; then JAVA_OPTS="-XX:+HeapDumpOnOutOfMemoryError -XX:HeapDumpPath=$OOM_DUMP_PATH" fi -JVM_ARGS=" -server \ - -Xmx${XMX_SIZE} \ - -Xms${XMX_SIZE} \ +SHUFFLE_SERVER_BASE_JVM_ARGS=${SHUFFLE_SERVER_BASE_JVM_ARGS:-" -server \ + -Xmx${SHUFFLE_SERVER_XMX_SIZE} \ + -Xms${SHUFFLE_SERVER_XMX_SIZE} \ ${MAX_DIRECT_MEMORY_OPTS} \ ${JAVA_OPTS} \ - -XX:+UseG1GC \ + -XX:+CrashOnOutOfMemoryError \ + -XX:+ExitOnOutOfMemoryError \ + -XX:+UnlockExperimentalVMOptions \ + -XX:+PrintCommandLineFlags"} + +DEFAULT_GC_ARGS=" -XX:+UseG1GC \ -XX:MaxGCPauseMillis=200 \ -XX:ParallelGCThreads=20 \ -XX:ConcGCThreads=5 \ -XX:InitiatingHeapOccupancyPercent=60 \ -XX:G1HeapRegionSize=32m \ - -XX:+UnlockExperimentalVMOptions \ - -XX:G1NewSizePercent=10 \ - -XX:+CrashOnOutOfMemoryError \ - -XX:+ExitOnOutOfMemoryError \ - -XX:+PrintCommandLineFlags" + -XX:G1NewSizePercent=10" GC_LOG_ARGS_LEGACY=" -XX:+PrintGC \ -XX:+PrintAdaptiveSizePolicy \ @@ -130,10 +132,10 @@ GC_LOG_ARGS_NEW=" -XX:+IgnoreUnrecognizedVMOptions \ -Xlog:gc+start=debug \ -Xlog:gc*:file=${RSS_LOG_DIR}/gc-%t.log:tags,uptime,time,level" -ARGS="" +JVM_LOG_ARGS="" if [ -f ${LOG_CONF_FILE} ]; then - ARGS="$ARGS -Dlog4j2.configurationFile=file:${LOG_CONF_FILE} -Dlog.path=${LOG_PATH} -Dshuffle.server.storage.audit.log.path=${SHUFFLE_SERVER_STORAGE_AUDIT_LOG_PATH} -Dshuffle.server.rpc.audit.log.path=${SHUFFLE_SERVER_RPC_AUDIT_LOG_PATH}" + JVM_LOG_ARGS=" -Dlog4j2.configurationFile=file:${LOG_CONF_FILE} -Dlog.path=${LOG_PATH} -Dshuffle.server.storage.audit.log.path=${SHUFFLE_SERVER_STORAGE_AUDIT_LOG_PATH} -Dshuffle.server.rpc.audit.log.path=${SHUFFLE_SERVER_RPC_AUDIT_LOG_PATH}" else echo "Exit with error: ${LOG_CONF_FILE} file doesn't exist." exit 1 @@ -141,13 +143,13 @@ fi version=$($RUNNER -version 2>&1 | awk -F[\".] '/version/ {print $2}') if [[ "$version" -lt "9" ]]; then - GC_ARGS=$GC_LOG_ARGS_LEGACY + SHUFFLE_SERVER_JVM_GC_ARGS="${SHUFFLE_SERVER_JVM_GC_ARGS:-${DEFAULT_GC_ARGS} ${GC_LOG_ARGS_LEGACY}}" else - GC_ARGS=$GC_LOG_ARGS_NEW + SHUFFLE_SERVER_JVM_GC_ARGS="${SHUFFLE_SERVER_JVM_GC_ARGS:-${DEFAULT_GC_ARGS} ${GC_LOG_ARGS_NEW}}" fi -UNIFFLE_SHUFFLE_SERVER_JAVA_OPTS=${UNIFFLE_SHUFFLE_SERVER_JAVA_OPTS:-""} -$RUNNER ${UNIFFLE_SHUFFLE_SERVER_JAVA_OPTS} $ARGS $JVM_ARGS $GC_ARGS $JAVA_LIB_PATH -cp $CLASSPATH $MAIN_CLASS --conf "$SHUFFLE_SERVER_CONF_FILE" $@ & +SHUFFLE_SERVER_JAVA_OPTS=${SHUFFLE_SERVER_JAVA_OPTS:-""} +$RUNNER ${SHUFFLE_SERVER_BASE_JVM_ARGS} ${SHUFFLE_SERVER_JVM_GC_ARGS} ${JVM_LOG_ARGS} ${JAVA_LIB_PATH} ${SHUFFLE_SERVER_JAVA_OPTS} -cp ${CLASSPATH} ${MAIN_CLASS} --conf "${SHUFFLE_SERVER_CONF_FILE}" $@ & get_pid_file_name shuffle-server echo $! >${RSS_PID_DIR}/${pid_file} diff --git a/conf/rss-env.sh b/conf/rss-env.sh index fb577e5d1..c505aadad 100644 --- a/conf/rss-env.sh +++ b/conf/rss-env.sh @@ -23,8 +23,10 @@ set -o errexit # exit the script if any statement returns a non-true return va # JAVA_HOME=<java_home_dir> # HADOOP_HOME=<hadoop_home_dir> -XMX_SIZE="80g" # Shuffle Server JVM XMX size +XMX_SIZE="8g" # Coordinator/Dashboard JVM XMX size +SHUFFLE_SERVER_XMX_SIZE="80g" # ShuffleServer JVM XMX size +# Variables for the all components # RSS_HOME, RSS home directory (Default: parent directory of the script). # If you want to set it to another place, please make sure the RSS_HOME # specified externally is respected, by defining it as follows: @@ -40,10 +42,24 @@ XMX_SIZE="80g" # Shuffle Server JVM XMX size # HADOOP_CONF_DIR, Hadoop configuration directory (Default: ${HADOOP_HOME}/etc/hadoop) # RSS_PID_DIR, Where the pid file is stored (Default: ${RSS_HOME}) # RSS_LOG_DIR, Where log files are stored (Default: ${RSS_HOME}/logs) + +# Variables for the dashboard +# DASHBOARD_BASE_JVM_ARGS, Base JVM options, which by default is calculated from +# `DASHBOARD_XMX_SIZE` or `XMX_SIZE and has sane defaults. Normally no need for customization. +# DASHBOARD_JAVA_OPTS, Extra JVM options for the dashboard, which will append after base arguments. +# This will override some default JVM options if they are both specified in default and this one. + +# Variables for the coordinator +# COORDINATOR_BASE_JVM_ARGS, Base JVM options, which by default is calculated from +# `COORDINATOR_XMX_SIZE` or `XMX_SIZE and has sane defaults. Normally no need for customization. +# COORDINATOR_JAVA_OPTS, Extra JVM options for the coordinator, which will append after base arguments. + +# Variables for the shuffle server # RSS_IP, IP address Shuffle Server binds to on this node (Default: first non-loopback ipv4) # MAX_DIRECT_MEMORY_SIZE Shuffle Server JVM off heap memory size (Default: not set) # MALLOC_ARENA_MAX, Set the number of memory arenas for Shuffle Server (Default: 4) - -# UNIFFLE_DASHBOARD_JAVA_OPTS, JVM options for the dashboard -# UNIFFLE_SHUFFLE_SERVER_JAVA_OPTS, JVM options for the shuffle server -# UNIFFLE_COORDINATOR_JAVA_OPTS, JVM options for the coordinator +# SHUFFLE_SERVER_JVM_ARGS, Base JVM options, which by default is calculated from +# `SHUFFLE_SERVER_XMX_SIZE` or `XMX_SIZE and has sane defaults. Normally no need for customization. +# SHUFFLE_SERVER_JAVA_OPTS, Extra JVM options for the shuffle server, which will append after base arguments. +# SHUFFLE_SERVER_STORAGE_AUDIT_LOG_PATH, Shuffle Server storage audit log path (Default: ${RSS_LOG_DIR}/shuffle_server_storage_audit.log) +# SHUFFLE_SERVER_RPC_AUDIT_LOG_PATH, Shuffle Server RPC audit log path (Default: ${RSS_LOG_DIR}/shuffle_server_rpc_audit.log) diff --git a/deploy/kubernetes/docker/rss-env.sh b/deploy/kubernetes/docker/rss-env.sh index 7c8715331..d6b0dfc48 100644 --- a/deploy/kubernetes/docker/rss-env.sh +++ b/deploy/kubernetes/docker/rss-env.sh @@ -25,6 +25,7 @@ HADOOP_HOME="/data/rssadmin/hadoop" RUNNER="${JAVA_HOME}/bin/java" JPS="${JAVA_HOME}/bin/jps" +# Variables for the all components # RSS_HOME, RSS home directory (Default: parent directory of the script). # If you want to set it to another place, please make sure the RSS_HOME # specified externally is respected, by defining it as follows: @@ -40,4 +41,24 @@ JPS="${JAVA_HOME}/bin/jps" # HADOOP_CONF_DIR, Hadoop configuration directory (Default: ${HADOOP_HOME}/etc/hadoop) # RSS_PID_DIR, Where the pid file is stored (Default: ${RSS_HOME}) # RSS_LOG_DIR, Where log files are stored (Default: ${RSS_HOME}/logs) + +# Variables for the dashboard +# DASHBOARD_BASE_JVM_ARGS, Base JVM options, which by default is calculated from +# `DASHBOARD_XMX_SIZE` or `XMX_SIZE and has sane defaults. Normally no need for customization. +# DASHBOARD_JAVA_OPTS, Extra JVM options for the dashboard, which will append after base arguments. +# This will override some default JVM options if they are both specified in default and this one. + +# Variables for the coordinator +# COORDINATOR_BASE_JVM_ARGS, Base JVM options, which by default is calculated from +# `COORDINATOR_XMX_SIZE` or `XMX_SIZE and has sane defaults. Normally no need for customization. +# COORDINATOR_JAVA_OPTS, Extra JVM options for the coordinator, which will append after base arguments. + +# Variables for the shuffle server # RSS_IP, IP address Shuffle Server binds to on this node (Default: first non-loopback ipv4) +# MAX_DIRECT_MEMORY_SIZE Shuffle Server JVM off heap memory size (Default: not set) +# MALLOC_ARENA_MAX, Set the number of memory arenas for Shuffle Server (Default: 4) +# SHUFFLE_SERVER_JVM_ARGS, Base JVM options, which by default is calculated from +# `SHUFFLE_SERVER_XMX_SIZE` or `XMX_SIZE and has sane defaults. Normally no need for customization. +# SHUFFLE_SERVER_JAVA_OPTS, Extra JVM options for the shuffle server, which will append after base arguments. +# SHUFFLE_SERVER_STORAGE_AUDIT_LOG_PATH, Shuffle Server storage audit log path (Default: ${RSS_LOG_DIR}/shuffle_server_storage_audit.log) +# SHUFFLE_SERVER_RPC_AUDIT_LOG_PATH, Shuffle Server RPC audit log path (Default: ${RSS_LOG_DIR}/shuffle_server_rpc_audit.log) diff --git a/deploy/kubernetes/operator/examples/configuration.yaml b/deploy/kubernetes/operator/examples/configuration.yaml index 6b7183b5f..e18fa6264 100644 --- a/deploy/kubernetes/operator/examples/configuration.yaml +++ b/deploy/kubernetes/operator/examples/configuration.yaml @@ -93,4 +93,12 @@ data: rss.server.read.buffer.capacity 5g rss.server.shuffle.expired.timeout.ms 120000 rss.server.write.retry.max 2 - rss.storage.type MEMORY_LOCALFILE \ No newline at end of file + rss.storage.type MEMORY_LOCALFILE + rss-env.sh: |- + #!/usr/bin/env bash + set -o pipefail + set -o nounset # exit the script if you try to use an uninitialised variable + set -o errexit # exit the script if any statement returns a non-true return value + HADOOP_HOME="/data/rssadmin/hadoop" + RUNNER="${JAVA_HOME}/bin/java" + JPS="${JAVA_HOME}/bin/jps" diff --git a/docs/coordinator_guide.md b/docs/coordinator_guide.md index 978e0a107..e443494c1 100644 --- a/docs/coordinator_guide.md +++ b/docs/coordinator_guide.md @@ -34,12 +34,12 @@ This document will introduce how to deploy Uniffle coordinators. ``` JAVA_HOME=<java_home> HADOOP_HOME=<hadoop home> - XMX_SIZE="16g" + COORDINATOR_XMX_SIZE="16g" ``` - You can add extra JVM arguments for the Uniffle coordinator by specifying `UNIFFLE_COORDINATOR_JAVA_OPTS` in `rss-env.sh`. + You can add extra JVM arguments for the Uniffle coordinator by specifying `COORDINATOR_JAVA_OPTS` in `rss-env.sh`. Example: ``` - UNIFFLE_COORDINATOR_JAVA_OPTS="-Xdebug -Xrunjdwp:transport=dt_socket,server=y,suspend=y,address=5006 -Drss.jetty.http.port=19998" + COORDINATOR_JAVA_OPTS="-Xdebug -Xrunjdwp:transport=dt_socket,server=y,suspend=y,address=5006 -Drss.jetty.http.port=19998" ``` 3. update RSS_HOME/conf/coordinator.conf, eg, ``` diff --git a/docs/dashboard_guide.md b/docs/dashboard_guide.md index 48a2a8e53..4f23c4037 100644 --- a/docs/dashboard_guide.md +++ b/docs/dashboard_guide.md @@ -33,10 +33,10 @@ coordinator.web.address http://coordinator.hostname:19998/ ``` ### Configure related JVM parameters -You can add extra JVM arguments for the Uniffle dashboard by specifying `UNIFFLE_DASHBOARD_JAVA_OPTS` in `RSS_HOME/bin/rss-env.sh` +You can add extra JVM arguments for the Uniffle dashboard by specifying `DASHBOARD_JAVA_OPTS` in `RSS_HOME/bin/rss-env.sh` Example: ``` -UNIFFLE_DASHBOARD_JAVA_OPTS="-Xdebug -Xrunjdwp:transport=dt_socket,server=y,suspend=y,address=5004 -Drss.jetty.http.port=19997" +DASHBOARD_JAVA_OPTS="-Xdebug -Xrunjdwp:transport=dt_socket,server=y,suspend=y,address=5004 -Drss.jetty.http.port=19997" ``` ### Start the dashboard process diff --git a/docs/server_guide.md b/docs/server_guide.md index 16e2146ff..3cc061c1e 100644 --- a/docs/server_guide.md +++ b/docs/server_guide.md @@ -30,7 +30,7 @@ This document will introduce how to deploy Uniffle shuffle servers. ``` JAVA_HOME=<java_home> HADOOP_HOME=<hadoop home> - XMX_SIZE="80g" + SHUFFLE_SERVER_XMX_SIZE="80g" ``` For the following cases, you don't need to specify `HADOOP_HOME` that will simplify the server deployment. @@ -176,12 +176,12 @@ XMX_SIZE=40g MAX_DIRECT_MEMORY_SIZE=360g ``` -You can add extra JVM arguments for the Uniffle server by specifying `UNIFFLE_SHUFFLE_SERVER_JAVA_OPTS` in `rss-env.sh`. +You can add extra JVM arguments for the Uniffle server by specifying `SHUFFLE_SERVER_JAVA_OPTS` in `rss-env.sh`. For example: ``` -UNIFFLE_SHUFFLE_SERVER_JAVA_OPTS="-Xdebug -Xrunjdwp:transport=dt_socket,server=y,suspend=y,address=5005 -Drss.jetty.http.port=19978" +SHUFFLE_SERVER_JAVA_OPTS="-Xdebug -Xrunjdwp:transport=dt_socket,server=y,suspend=y,address=5005 -Drss.jetty.http.port=19978" ``` ##### server.conf