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-22852][BUILD] Exclude -Xlint:unchecked from sbt javadoc flags #20040

Closed
wants to merge 1 commit into from

Conversation

easel
Copy link
Contributor

@easel easel commented Dec 20, 2017

What changes were proposed in this pull request?

Moves the -Xlint:unchecked flag in the sbt build configuration from Compile to (Compile, compile) scope, allowing publish and publishLocal commands to work.

How was this patch tested?

Successfully published the spark-launcher subproject from within sbt successfully, where it fails without this patch.

@vanzin
Copy link
Contributor

vanzin commented Dec 20, 2017

ok to test

@SparkQA
Copy link

SparkQA commented Dec 21, 2017

Test build #85213 has finished for PR 20040 at commit 1827c2d.

  • This patch fails PySpark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Dec 21, 2017

Test build #4020 has finished for PR 20040 at commit 1827c2d.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@HyukjinKwon
Copy link
Member

retest this please

@SparkQA
Copy link

SparkQA commented Dec 21, 2017

Test build #85257 has finished for PR 20040 at commit 1827c2d.

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

@srowen
Copy link
Member

srowen commented Dec 21, 2017

Merged to master

@asfgit asfgit closed this in 0abaf31 Dec 21, 2017
@easel easel deleted the javadoc-xlint branch December 21, 2017 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants