Change premerge to build all Spark versions #3631
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Thomas Graves <[email protected]>
tgravescs
requested review from
GaryShen2008,
jlowe and
NvTimLiu
as code owners
|
build |
gerashegalov
previously approved these changes
Sep 23, 2021
razajafri
previously approved these changes
Sep 23, 2021
jlowe
previously approved these changes
Sep 23, 2021
|
premerge failed due to cdh compile issue, I'll wait til that is fixed and retest this |
|
build |
jenkins/spark-premerge-build.sh
Outdated
| env -u SPARK_HOME mvn -U -B $MVN_URM_MIRROR -Dbuildver=312 clean install -Drat.skip=true -DskipTests -Dmaven.javadoc.skip=true -Dskip -Dmaven.scalastyle.skip=true -Dcuda.version=$CUDA_CLASSIFIER -pl aggregator -am | ||
| env -u SPARK_HOME mvn -U -B $MVN_URM_MIRROR -Dbuildver=313 clean install -Drat.skip=true -DskipTests -Dmaven.javadoc.skip=true -Dskip -Dmaven.scalastyle.skip=true -Dcuda.version=$CUDA_CLASSIFIER -pl aggregator -am | ||
| # don't skip tests | ||
| env -u SPARK_HOME mvn -U -B $MVN_URM_MIRROR -Dbuildver=320 clean install -Drat.skip=true -Dmaven.javadoc.skip=true -Dskip -Dmaven.scalastyle.skip=true -Dcuda.version=$CUDA_CLASSIFIER -pl -Dpytest.TEST_TAGS='' |
There was a problem hiding this comment.
CI catches aggregator is missing here
There was a problem hiding this comment.
actually removed -pl for this one because running tests with it so want to build everything
tgravescs
dismissed stale reviews from jlowe, razajafri, and gerashegalov
via
996c132
|
build |
gerashegalov
previously approved these changes
Sep 23, 2021
|
switch the profile back to individual from snapshots. somehow it appears to be downloading jars to the deps directory with timestamp for cdh jar instead of snapshot, so I'll followup with that separately. |
|
build |
gerashegalov
approved these changes
Sep 24, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Build all the versions to catch bugs in not updating shims.