Skip to content

[SPARK-24685][BUILD][FOLLOWUP] Fix the nonexist profile name in release script#22434

Closed
jerryshao wants to merge 1 commit intoapache:masterfrom
jerryshao:SPARK-24685-followup
Closed

[SPARK-24685][BUILD][FOLLOWUP] Fix the nonexist profile name in release script#22434
jerryshao wants to merge 1 commit intoapache:masterfrom
jerryshao:SPARK-24685-followup

Conversation

@jerryshao
Copy link
Contributor

What changes were proposed in this pull request?

without-hadoop profile doesn't exist in Maven, instead the name should be hadoop-provided, this is a regression introduced by SPARK-24685. So here fix it.

How was this patch tested?

Local test.

@jerryshao
Copy link
Contributor Author

@cloud-fan @vanzin , please help to review, thanks!

@cloud-fan
Copy link
Contributor

LGTM

@SparkQA
Copy link

SparkQA commented Sep 16, 2018

Test build #96107 has finished for PR 22434 at commit 18a9135.

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

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM.

@cloud-fan
Copy link
Contributor

thanks, merging to master/2.4!

asfgit pushed a commit that referenced this pull request Sep 17, 2018
…se script

## What changes were proposed in this pull request?

`without-hadoop` profile doesn't exist in Maven, instead the name should be `hadoop-provided`, this is a regression introduced by SPARK-24685. So here fix it.

## How was this patch tested?

Local test.

Closes #22434 from jerryshao/SPARK-24685-followup.

Authored-by: jerryshao <sshao@hortonworks.com>
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
(cherry picked from commit b66e14d)
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
@asfgit asfgit closed this in b66e14d Sep 17, 2018
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.

5 participants