Github user erikerlandson commented on a diff in the pull request: https://github.com/apache/spark/pull/21279#discussion_r213317184 --- Diff: bin/docker-image-tool.sh --- @@ -44,15 +44,37 @@ function image_ref { function build { local BUILD_ARGS local IMG_PATH + local TMPFOLDER if [ ! -f "$SPARK_HOME/RELEASE" ]; then # Set image build arguments accordingly if this is a source repo and not a distribution archive. + local JARS="${SPARK_HOME}/assembly/target/scala-${SPARK_SCALA_VERSION}/jars" + TMPFOLDER=`mktemp -q -d examples.XXXXXX` + if [ $? -ne 0 ]; then + ehco "Cannot create temp folder, exiting..." + exit 1 + fi + + mkdir -p "${TMPFOLDER}/jars" --- End diff -- Also, is using `${SPARK_SCALA_VERSION}` preferable to `scala*` ?
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org