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-29528][BUILD][test-maven] Upgrade scala-maven-plugin to 4.2.4 for Scala 2.13.1 #26185

Closed
wants to merge 1 commit into from
Closed

Conversation

dongjoon-hyun
Copy link
Member

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

What changes were proposed in this pull request?

This PR upgrades scala-maven-plugin to 4.2.4 for Scala 2.13.1.

Why are the changes needed?

Scala 2.13.1 seems to break the binary compatibility.

We need to upgrade scala-maven-plugin to bring the the following fixes for the latest Scala 2.13.1.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

For now, we don't support Scala-2.13. This PR at least needs to pass the existing Jenkins with Maven to get prepared for Scala-2.13.

@dongjoon-hyun dongjoon-hyun changed the title [SPARK-29528][BUILD] Upgrade scala-maven-plugin to 4.2.4 for Scala 2.13.1 [SPARK-29528][BUILD][test-maven] Upgrade scala-maven-plugin to 4.2.4 for Scala 2.13.1 Oct 20, 2019
@dongjoon-hyun
Copy link
Member Author

Retest this please.

@SparkQA
Copy link

SparkQA commented Oct 21, 2019

Test build #112351 has finished for PR 26185 at commit 807bcde.

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

@SparkQA
Copy link

SparkQA commented Oct 21, 2019

Test build #112352 has finished for PR 26185 at commit 807bcde.

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

@HyukjinKwon
Copy link
Member

Merged to master.

Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

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

LGTM

@dongjoon-hyun
Copy link
Member Author

Thank you for review and merging, @HyukjinKwon !

@dongjoon-hyun dongjoon-hyun deleted the SPARK-29528 branch October 21, 2019 16:09
@HyukjinKwon
Copy link
Member

Ah .. sorry @dongjoon-hyun, this seems causing the build failure on Windows (#26278). Let me revert this one for preview, and then get this in after the preview just to make sure all tests pass.

@dongjoon-hyun
Copy link
Member Author

No problem~ Thanks!

@dongjoon-hyun
Copy link
Member Author

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.

3 participants