Skip to content

Conversation

@wangyum
Copy link
Member

@wangyum wangyum commented Mar 10, 2019

What changes were proposed in this pull request?

Add test-hadoop3.1 phrase to test Spark against Spark’s Hadoop 3.1 profile.

How was this patch tested?

Tested on jenkins. This is output:

[info] Using build tool sbt with Hadoop profile hadoop3.1 under environment amplab_jenkins
...
[info] Building Spark (w/Hive 1.2.1) using SBT with these arguments:  -Phadoop-3.1 -Pkubernetes -Phive-thriftserver -Pkinesis-asl -Pyarn -Pspark-ganglia-lgpl -Phive -Pmesos test:package streaming-kinesis-asl-assembly/assembly

https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/103282/console

Copy link
Member

@felixcheung felixcheung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doesn't riselab jenkins need to be configured to set test-hadoop3.1?

@SparkQA
Copy link

SparkQA commented Mar 10, 2019

Test build #103282 has finished for PR 24045 at commit 081f559.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@HyukjinKwon
Copy link
Member

It does to build regularly like spark-branch-2.4-test-sbt-hadoop-2.7. Looks this targets to extract profile from PR title, which can be also used in PR builder. Hadoop 3 profile tests cannot be passed for now.

@HyukjinKwon
Copy link
Member

Merged to master, to unblock @wangyum's work.

@HyukjinKwon
Copy link
Member

@wangyum, BTW, let's file a separate JIRA next time when the fixed versions are expected to be different.

@wangyum
Copy link
Member Author

wangyum commented Mar 11, 2019

Got it. Thanks @HyukjinKwon
This PR very similar to SPARK-12842. we still need update release-build.sh and building-spark.md once we officially support Hadoop-3.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants