Skip to content

[SPARK-35268][BUILD] Upgrade GenJavadoc to 0.17#32392

Closed
sarutak wants to merge 1 commit intoapache:masterfrom
sarutak:upgrade-genjavadoc-0.17
Closed

[SPARK-35268][BUILD] Upgrade GenJavadoc to 0.17#32392
sarutak wants to merge 1 commit intoapache:masterfrom
sarutak:upgrade-genjavadoc-0.17

Conversation

@sarutak
Copy link
Member

@sarutak sarutak commented Apr 29, 2021

What changes were proposed in this pull request?

This PR upgrades GenJavadoc to 0.17.

Why are the changes needed?

This version seems to include a fix for an issue which can happen with Scala 2.13.5.
https://github.com/lightbend/genjavadoc/releases/tag/v0.17

Does this PR introduce any user-facing change?

No.

How was this patch tested?

I confirmed build succeed with the following commands.

# For Scala 2.12
$ build/sbt -Phive -Phive-thriftserver -Pyarn -Pmesos -Pkubernetes -Phadoop-cloud -Pspark-ganglia-lgpl -Pkinesis-asl -Pdocker-integration-tests -Pkubernetes-integration-tests unidoc

# For Scala 2.13
build/sbt -Phive -Phive-thriftserver -Pyarn -Pmesos -Pkubernetes -Phadoop-cloud -Pspark-ganglia-lgpl -Pkinesis-asl -Pdocker-integration-tests -Pkubernetes-integration-tests -Pscala-2.13 unidoc

@github-actions github-actions bot added the BUILD label Apr 29, 2021
@SparkQA
Copy link

SparkQA commented Apr 29, 2021

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

@SparkQA
Copy link

SparkQA commented Apr 29, 2021

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

@SparkQA
Copy link

SparkQA commented Apr 29, 2021

Test build #138070 has finished for PR 32392 at commit 60b9a23.

  • 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. Thank you, @sarutak and @srowen .
Merged to master.

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.

4 participants