GitHub user JoshRosen opened a pull request:
https://github.com/apache/spark/pull/10704
[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
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/JoshRosen/spark fix-build-test-problems
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/10704.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #10704
----
commit ca84f78ed0f5495d428e41251016ed74849fdd27
Author: Josh Rosen <[email protected]>
Date: 2016-01-11T20:11:00Z
Make build changes trigger all tests
commit e99b34a2fe0cac4f135d25bf310cc21cfabdd838
Author: Josh Rosen <[email protected]>
Date: 2016-01-11T20:16:22Z
Run clean after installing dummy JARs in dev/test-dependencies.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
