[CELEBORN-838] Add custom mvn flag to celeborn

### What changes were proposed in this pull request? Add an option to pass in a custom maven installation, similar to how [Spark does it](https://github.com/apache/spark/blob/master/dev/make-distribution.sh#L65). ### Why are the changes needed? We need this internally as some of our machines may not have access to external Maven. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? ran make-distribution.sh to make sure it worked. Closes #1761 from akpatnam25/CELEBORN-838. Authored-by: Aravind Patnam <[email protected]> Signed-off-by: Cheng Pan <[email protected]>
apache · Jul 26, 2023 · e708c3c · e708c3c
1 parent e16b267
commit e708c3c
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/build/make-distribution.sh b/build/make-distribution.sh
@@ -24,12 +24,13 @@ PROJECT_DIR="$(cd "`dirname "$0"`/.."; pwd)"
 DIST_DIR="$PROJECT_DIR/dist"
 NAME="bin"
 RELEASE="false"
+MVN="$PROJECT_DIR/build/mvn"
 
 function exit_with_usage {
   echo "make-distribution.sh - tool for making binary distributions of Celeborn"
   echo ""
   echo "usage:"
-  cl_options="[--name <custom_name>] [--release]"
+  cl_options="[--name <custom_name>] [--release] [--mvn <mvn-command>]"
   echo "make-distribution.sh $cl_options <maven build options>"
   echo ""
   exit 1
@@ -42,6 +43,10 @@ while (( "$#" )); do
       NAME="bin-$2"
       shift
       ;;
+    --mvn)
+      MVN="$2"
+      shift
+      ;;
     --release)
       RELEASE="true"
       ;;
@@ -86,7 +91,6 @@ if [ -z "$JAVA_HOME" ]; then
   exit -1
 fi
 
-MVN="$PROJECT_DIR/build/mvn"
 export MAVEN_OPTS="${MAVEN_OPTS:--Xmx2g -XX:ReservedCodeCacheSize=1g}"
 
 if [ ! "$(command -v "$MVN")" ] ; then