Skip to content

[SPARK-32511][SQL][FOLLOWUP] Fix the broken build for Scala 2.13 with Maven #29954

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.

Sign up for GitHub

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

Jump to bottom
Closed
sarutak wants to merge 2 commits into from
Closed

[SPARK-32511][SQL][FOLLOWUP] Fix the broken build for Scala 2.13 with Maven #29954

sarutak wants to merge 2 commits into from

Conversation

@sarutak
Copy link
Member

@sarutak sarutak commented Oct 6, 2020 •
edited
Loading

What changes were proposed in this pull request?

This PR fixes the broken build for Scala 2.13 with Maven.
https://github.com/apache/spark/pull/29913/checks?check_run_id=1187826966

#29795 was merged though it doesn't successfully finish the build for Scala 2.13

Why are the changes needed?

To fix the build.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

build/mvn -Pscala-2.13 -Phive -Phive-thriftserver -DskipTests package

@SparkQA
Copy link

SparkQA commented Oct 6, 2020

Test build #129447 has finished for PR 29954 at commit 69846d0.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@sarutak
Copy link
Member Author

sarutak commented Oct 6, 2020

This compilation error will be fixed by #29955 .

@sarutak
Copy link
Member Author

sarutak commented Oct 6, 2020

retest this please.

@sarutak
Copy link
Member Author

sarutak commented Oct 6, 2020

cc: @srowen

@SparkQA
Copy link

SparkQA commented Oct 6, 2020

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34054/

@SparkQA
Copy link

SparkQA commented Oct 6, 2020

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34054/

@SparkQA
Copy link

SparkQA commented Oct 6, 2020

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34056/

@SparkQA
Copy link

SparkQA commented Oct 6, 2020

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34058/

@SparkQA
Copy link

SparkQA commented Oct 6, 2020

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34056/

@SparkQA
Copy link

SparkQA commented Oct 6, 2020

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34058/

@dongjoon-hyun dongjoon-hyun changed the title [HOTFIX][BUILD] Fix the broken build for Scala 2.13 with Maven [SPARK-32511][SQL][FOLLOWUP] Fix the broken build for Scala 2.13 with Maven Oct 6, 2020
@dongjoon-hyun
Copy link
Member

dongjoon-hyun commented Oct 6, 2020

+1, LGTM. Thank you, @sarutak and @srowen .

@SparkQA
Copy link

SparkQA commented Oct 6, 2020

Test build #129449 has finished for PR 29954 at commit 69846d0.

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

@SparkQA
Copy link

SparkQA commented Oct 6, 2020

Test build #129451 has finished for PR 29954 at commit 655e0a7.

  • 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

Reviewers

@srowen srowen srowen approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sarutak @SparkQA @dongjoon-hyun @srowen