Skip to content

Conversation

@dongjoon-hyun
Copy link
Member

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

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
Member Author

cc @cloud-fan , @dbtsai , @HyukjinKwon

Copy link
Contributor

@cloud-fan cloud-fan left a comment

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
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 <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
@dongjoon-hyun dongjoon-hyun deleted the hotfix branch March 27, 2020 08:01
@HyukjinKwon
Copy link
Member

LGTM thanks @dongjoon-hyun for resolving the conflicts.

@SparkQA
Copy link

SparkQA commented Mar 27, 2020

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
Copy link
Member

dbtsai commented Mar 27, 2020

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