Github user ramaddepally commented on a diff in the pull request:
https://github.com/apache/spark/pull/23053#discussion_r234335016
--- Diff: bin/docker-image-tool.sh ---
@@ -102,33 +114,33 @@ function build {
error "Failed to build Spark JVM Docker image, please refer to Docker
build output for details."
fi
- docker build $NOCACHEARG "${BINDING_BUILD_ARGS[@]}" \
- -t $(image_ref spark-py) \
- -f "$PYDOCKERFILE" .
+ if [ "${PYDOCKERFILE}" != "skip" ]; then
+ docker build $NOCACHEARG "${BINDING_BUILD_ARGS[@]}" \
+ -t $(image_ref spark-py) \
+ -f "$PYDOCKERFILE" .
+ if [ $? -ne 0 ]; then
+ error "Failed to build PySpark Docker image, please refer to
Docker build output for details."
+ fi
+ else
+ echo "Skipped building PySpark docker image."
+ fi
+
+ if [ "${RDOCKERFILE}" != "skip" ] && [ -d "${SPARK_HOME}/R/lib" ]; then
+ docker build $NOCACHEARG "${BINDING_BUILD_ARGS[@]}" \
+ -t $(image_ref spark-r) \
+ -f "$RDOCKERFILE" .
if [ $? -ne 0 ]; then
- error "Failed to build PySpark Docker image, please refer to Docker
build output for details."
+ error "Failed to build SparkR Docker image, please refer to Docker
build output for details."
fi
- docker build $NOCACHEARG "${BINDING_BUILD_ARGS[@]}" \
- -t $(image_ref spark-r) \
- -f "$RDOCKERFILE" .
- if [ $? -ne 0 ]; then
- error "Failed to build SparkR Docker image, please refer to Docker
build output for details."
+ else
+ echo "Skipped building SparkR docker image."
--- End diff --
Added explicit message when building image is skipped due to R artifacts
not found.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
