Skip to content

Commit 7c0ce28

Browse files
dongjoon-hyunHyukjinKwon
authored andcommitted
[SPARK-30056][INFRA] Skip building test artifacts in dev/make-distribution.sh
### What changes were proposed in this pull request? This PR aims to skip building test artifacts in `dev/make-distribution.sh`. Since Apache Spark 3.0.0, we need to build additional binary distribution, this helps the release process by speeding up building multiple binary distributions. ### Why are the changes needed? Since the generated binary artifacts are irrelevant to the test jars, we can skip this. **BEFORE** ``` $ time dev/make-distribution.sh 726.86 real 2526.04 user 45.63 sys ``` **AFTER** ``` $ time dev/make-distribution.sh 305.54 real 1099.99 user 26.52 sys ``` ### Does this PR introduce any user-facing change? No. ### How was this patch tested? Manually check `dev/make-distribution.sh` result and time. Closes #26689 from dongjoon-hyun/SPARK-30056. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: HyukjinKwon <[email protected]>
1 parent 19af1fe commit 7c0ce28

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

dev/make-distribution.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@ export MAVEN_OPTS="${MAVEN_OPTS:--Xmx2g -XX:ReservedCodeCacheSize=1g}"
165165
# Store the command as an array because $MVN variable might have spaces in it.
166166
# Normal quoting tricks don't work.
167167
# See: http://mywiki.wooledge.org/BashFAQ/050
168-
BUILD_COMMAND=("$MVN" clean package -DskipTests $@)
168+
BUILD_COMMAND=("$MVN" clean package -DskipTests -Dmaven.test.skip=true $@)
169169

170170
# Actually build the jar
171171
echo -e "\nBuilding with..."

0 commit comments

Comments
 (0)