Skip to content

[SPARK-25556][SPARK-17636][SPARK-31026][SPARK-31060][FOLLOWUP][3.0] Fix build error due to conf version - #28046

Closed
dongjoon-hyun wants to merge 1 commit into
apache:branch-3.0from
dongjoon-hyun:hotfix
Closed

[SPARK-25556][SPARK-17636][SPARK-31026][SPARK-31060][FOLLOWUP][3.0] Fix build error due to conf version#28046
dongjoon-hyun wants to merge 1 commit into
apache:branch-3.0from
dongjoon-hyun:hotfix

Conversation

@dongjoon-hyun

@dongjoon-hyun dongjoon-hyun commented Mar 27, 2020

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

This fixes the build error.

Why are the changes needed?

branch-3.0 doesn't have conf version.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Passing the GitHub Action is enough.

@dongjoon-hyun dongjoon-hyun changed the title [SPARK-25556][SPARK-17636][SPARK-31026][SPARK-31060][FOLLOWUP][3.0] F… [SPARK-25556][SPARK-17636][SPARK-31026][SPARK-31060][FOLLOWUP][3.0] Fix build error due to conf version Mar 27, 2020
@dongjoon-hyun

Copy link
Copy Markdown
Member Author

cc @cloud-fan , @dbtsai , @HyukjinKwon

@cloud-fan cloud-fan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks for fixing!

@dongjoon-hyun

Copy link
Copy Markdown
Member Author

Thank you, @cloud-fan .
Since GitHub Action passed, I'll merge this to branch-3.0.

dongjoon-hyun added a commit that referenced this pull request Mar 27, 2020
…ix build error due to conf version

### What changes were proposed in this pull request?

This fixes the build error.

### Why are the changes needed?

`branch-3.0` doesn't have conf version.

### Does this PR introduce any user-facing change?

No.

### How was this patch tested?

Passing the GitHub Action is enough.

Closes #28046 from dongjoon-hyun/hotfix.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
@dongjoon-hyun
dongjoon-hyun deleted the hotfix branch March 27, 2020 08:01
@HyukjinKwon

Copy link
Copy Markdown
Member

LGTM thanks @dongjoon-hyun for resolving the conflicts.

@SparkQA

SparkQA commented Mar 27, 2020

Copy link
Copy Markdown

Test build #120461 has finished for PR 28046 at commit 62201b0.

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

@dbtsai

dbtsai commented Mar 27, 2020

Copy link
Copy Markdown
Member

Thanks @dongjoon-hyun !!!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants