Skip to content

Conversation

@thunterdb
Copy link
Contributor

Currently, trying to publish spark locally fails when java version is >= 1.8.0:

  • a javadoc option has been removed
  • javadoc is stricter about its input and fails with some malformed comments

This PR fixes both issues.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am not sure how to get the current version of java (not the target version).

@JoshRosen
Copy link
Contributor

@thunterdb, I have a different fix for the Javadoc compilation issue which doesn't completely remove the flag. PR pending.

@SparkQA
Copy link

SparkQA commented Dec 1, 2015

Test build #46902 has finished for PR 10048 at commit 9900539.

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

asfgit pushed a commit that referenced this pull request Dec 1, 2015
This pull request fixes multiple issues with API doc generation.

- Modify the Jekyll plugin so that the entire doc build fails if API docs cannot be generated. This will make it easy to detect when the doc build breaks, since this will now trigger Jenkins failures.
- Change how we handle the `-target` compiler option flag in order to fix `javadoc` generation.
- Incorporate doc changes from thunterdb (in #10048).

Closes #10048.

Author: Josh Rosen <[email protected]>
Author: Timothy Hunter <[email protected]>

Closes #10049 from JoshRosen/fix-doc-build.

(cherry picked from commit d3ca8cf)
Signed-off-by: Josh Rosen <[email protected]>
@asfgit asfgit closed this in d3ca8cf Dec 1, 2015
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.

3 participants