Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-29608][BUILD] Add hadoop-3.2 profile to release build #26260

Closed
wants to merge 1 commit into from
Closed

[SPARK-29608][BUILD] Add hadoop-3.2 profile to release build #26260

wants to merge 1 commit into from

Conversation

dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Oct 25, 2019

What changes were proposed in this pull request?

This PR aims to add hadoop-3.2 profile to pre-built binary package releases.

Why are the changes needed?

Since Apache Spark 3.0.0, we provides Hadoop 3.2 pre-built binary.

Does this PR introduce any user-facing change?

No. (Although the artifacts are available, this change is for release managers).

How was this patch tested?

Manual. Please note that DRY_RUN=0 disables these combination.

$ dev/create-release/release-build.sh package
...
Packages to build: without-hadoop hadoop3.2 hadoop2.7
make_binary_release without-hadoop -Pscala-2.12 -Phadoop-provided  2.12
make_binary_release hadoop3.2 -Pscala-2.12 -Phadoop-3.2 -Phive -Phive-thriftserver  2.12
make_binary_release hadoop2.7 -Pscala-2.12 -Phadoop-2.7 -Phive -Phive-thriftserver withpip,withr 2.12

@dongjoon-hyun
Copy link
Member Author

Hi, @srowen and @jiangxb1987 .
Could you review this?

@dongjoon-hyun
Copy link
Member Author

Thank you for review and approval. Since this is a release script, Jenkins run is irrelevant to this PR.
I'll merge this.

Merged to master.

@dongjoon-hyun dongjoon-hyun reopened this Oct 25, 2019
@dongjoon-hyun dongjoon-hyun deleted the SPARK-29608 branch October 25, 2019 20:58
@SparkQA
Copy link

SparkQA commented Oct 25, 2019

Test build #112689 has finished for PR 26260 at commit 5f96af8.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants