Skip to content

Commit b8c6d93

Browse files
andrushaMarcelo Vanzin
authored andcommitted
[SPARK-23133][K8S] Fix passing java options to Executor
Pass through spark java options to the executor in context of docker image. Closes apache#20296 andrusha: Deployed two version of containers to local k8s, checked that java options were present in the updated image on the running executor. Manual test Author: Andrew Korzhuev <[email protected]> Closes apache#20322 from foxish/patch-1. (cherry picked from commit f568e9c) Signed-off-by: Marcelo Vanzin <[email protected]>
1 parent 1f88fcd commit b8c6d93

File tree

1 file changed

+3
-3
lines changed
  • resource-managers/kubernetes/docker/src/main/dockerfiles/spark

1 file changed

+3
-3
lines changed

resource-managers/kubernetes/docker/src/main/dockerfiles/spark/entrypoint.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ shift 1
4242

4343
SPARK_CLASSPATH="$SPARK_CLASSPATH:${SPARK_HOME}/jars/*"
4444
env | grep SPARK_JAVA_OPT_ | sed 's/[^=]*=\(.*\)/\1/g' > /tmp/java_opts.txt
45-
readarray -t SPARK_DRIVER_JAVA_OPTS < /tmp/java_opts.txt
45+
readarray -t SPARK_JAVA_OPTS < /tmp/java_opts.txt
4646
if [ -n "$SPARK_MOUNTED_CLASSPATH" ]; then
4747
SPARK_CLASSPATH="$SPARK_CLASSPATH:$SPARK_MOUNTED_CLASSPATH"
4848
fi
@@ -54,7 +54,7 @@ case "$SPARK_K8S_CMD" in
5454
driver)
5555
CMD=(
5656
${JAVA_HOME}/bin/java
57-
"${SPARK_DRIVER_JAVA_OPTS[@]}"
57+
"${SPARK_JAVA_OPTS[@]}"
5858
-cp "$SPARK_CLASSPATH"
5959
-Xms$SPARK_DRIVER_MEMORY
6060
-Xmx$SPARK_DRIVER_MEMORY
@@ -67,7 +67,7 @@ case "$SPARK_K8S_CMD" in
6767
executor)
6868
CMD=(
6969
${JAVA_HOME}/bin/java
70-
"${SPARK_EXECUTOR_JAVA_OPTS[@]}"
70+
"${SPARK_JAVA_OPTS[@]}"
7171
-Xms$SPARK_EXECUTOR_MEMORY
7272
-Xmx$SPARK_EXECUTOR_MEMORY
7373
-cp "$SPARK_CLASSPATH"

0 commit comments

Comments
 (0)