Github user skonto commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21462#discussion_r193985575
  
    --- Diff: 
resource-managers/kubernetes/docker/src/main/dockerfiles/spark/entrypoint.sh ---
    @@ -45,12 +45,10 @@ shift 1
     
     SPARK_CLASSPATH="$SPARK_CLASSPATH:${SPARK_HOME}/jars/*"
     env | grep SPARK_JAVA_OPT_ | sort -t_ -k4 -n | sed 's/[^=]*=\(.*\)/\1/g' > 
/tmp/java_opts.txt
    -readarray -t SPARK_JAVA_OPTS < /tmp/java_opts.txt
    -if [ -n "$SPARK_MOUNTED_CLASSPATH" ]; then
    -  SPARK_CLASSPATH="$SPARK_CLASSPATH:$SPARK_MOUNTED_CLASSPATH"
    -fi
    -if [ -n "$SPARK_MOUNTED_FILES_DIR" ]; then
    -  cp -R "$SPARK_MOUNTED_FILES_DIR/." .
    +readarray -t SPARK_EXECUTOR_JAVA_OPTS < /tmp/java_opts.txt
    +
    +if [ -n "$SPARK_EXTRA_CLASSPATH" ]; then
    --- End diff --
    
    This var exists in docs but not in code.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to