DSP-19969 allow spark executor log to separate file

riptano · Dec 6, 2019 · edb11ec · edb11ec
1 parent 4ad6b43
commit edb11ec
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 3 deletions.
diff --git a/bin/manager_start.sh b/bin/manager_start.sh
@@ -27,15 +27,22 @@ JAVA_OPTS="-XX:MaxDirectMemorySize=$MAX_DIRECT_MEMORY
 
 MAIN="spark.jobserver.JobManager"
 
+if [ "$SEPARATE_EXECUTOR_LOGGING" = "true" ]
+then
+  EXECUTOR_SETTING=""
+else
+  EXECUTOR_SETTING="--conf spark.executor.extraJavaOptions=$LOGGING_OPTS"
+fi
+
 if [ ! -z $3 ]; then
   cmd='$SPARK_HOME/bin/spark-submit --class $MAIN --driver-memory $JOBSERVER_MEMORY
-  --conf "spark.executor.extraJavaOptions=$LOGGING_OPTS"
+  $EXECUTOR_SETTING
   --proxy-user $3
   --driver-java-options "$GC_OPTS $JAVA_OPTS $LOGGING_OPTS $CONFIG_OVERRIDES"
   $appdir/spark-job-server.jar $1 $2 $conffile'
 else
   cmd='$SPARK_HOME/bin/spark-submit --class $MAIN --driver-memory $JOBSERVER_MEMORY
-  --conf "spark.executor.extraJavaOptions=$LOGGING_OPTS"
+  $EXECUTOR_SETTING
   --driver-java-options "$GC_OPTS $JAVA_OPTS $LOGGING_OPTS $CONFIG_OVERRIDES"
   $appdir/spark-job-server.jar $1 $2 $conffile'
 fi

diff --git a/bin/server_start.sh b/bin/server_start.sh
@@ -43,8 +43,15 @@ if [ -f "$PIDFILE" ] && kill -0 $(cat "$PIDFILE"); then
    exit 1
 fi
 
+if [ "$SEPARATE_EXECUTOR_LOGGING" = "true" ]
+then
+  EXECUTOR_SETTING=""
+else
+  EXECUTOR_SETTING="--conf spark.executor.extraJavaOptions=$LOGGING_OPTS"
+fi
+
 cmd='$SPARK_HOME/bin/spark-submit --class $MAIN --driver-memory $JOBSERVER_MEMORY
-  --conf "spark.executor.extraJavaOptions=$LOGGING_OPTS"
+  $EXECUTOR_SETTING
   --driver-java-options "$GC_OPTS $JAVA_OPTS $LOGGING_OPTS $CONFIG_OVERRIDES"
   $@ $appdir/spark-job-server.jar $conffile'