diff options
author | Josh Rosen <joshrosen@databricks.com> | 2016-01-11 12:56:43 -0800 |
---|---|---|
committer | Josh Rosen <joshrosen@databricks.com> | 2016-01-11 12:56:43 -0800 |
commit | a44991453a43615028083ba9546f5cd93112f6bd (patch) | |
tree | 7892c2a789e9ec12bdba1732e597ecbfcfc31d21 /dev/test-dependencies.sh | |
parent | b313badaa049f847f33663c61cd70ee2f2cbebac (diff) | |
download | spark-a44991453a43615028083ba9546f5cd93112f6bd.tar.gz spark-a44991453a43615028083ba9546f5cd93112f6bd.tar.bz2 spark-a44991453a43615028083ba9546f5cd93112f6bd.zip |
[SPARK-12734][HOTFIX] Build changes must trigger all tests; clean after install in dep tests
This patch fixes a build/test issue caused by the combination of #10672 and a latent issue in the original `dev/test-dependencies` script.
First, changes which _only_ touched build files were not triggering full Jenkins runs, making it possible for a build change to be merged even though it could cause failures in other tests. The `root` build module now depends on `build`, so all tests will now be run whenever a build-related file is changed.
I also added a `clean` step to the Maven install step in `dev/test-dependencies` in order to address an issue where the dummy JARs stuck around and caused "multiple assembly JARs found" errors in tests.
/cc zsxwing
Author: Josh Rosen <joshrosen@databricks.com>
Closes #10704 from JoshRosen/fix-build-test-problems.
Diffstat (limited to 'dev/test-dependencies.sh')
-rwxr-xr-x | dev/test-dependencies.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/dev/test-dependencies.sh b/dev/test-dependencies.sh index def87aa408..3cb5d2be2a 100755 --- a/dev/test-dependencies.sh +++ b/dev/test-dependencies.sh @@ -70,7 +70,7 @@ $MVN -q versions:set -DnewVersion=$TEMP_VERSION -DgenerateBackupPoms=false > /de # Generate manifests for each Hadoop profile: for HADOOP_PROFILE in "${HADOOP_PROFILES[@]}"; do echo "Performing Maven install for $HADOOP_PROFILE" - $MVN $HADOOP2_MODULE_PROFILES -P$HADOOP_PROFILE jar:jar jar:test-jar install:install -q + $MVN $HADOOP2_MODULE_PROFILES -P$HADOOP_PROFILE jar:jar jar:test-jar install:install clean -q echo "Performing Maven validate for $HADOOP_PROFILE" $MVN $HADOOP2_MODULE_PROFILES -P$HADOOP_PROFILE validate -q |