Github user ueshin commented on the pull request:
https://github.com/apache/spark/pull/3058#issuecomment-61598539
@srowen, Thank you for your comment.
The answers are:
- Yes, there are 3 errors and 63 warnings and the errors, which are usage
of HTML entities issues, are fixable and the warnings, which are mainly missing
of Javadoc annotations like `@param` or `@return`, are fixable if developers
write Javadocs correctly.
- The checks are grouped and we can choose which groups should be checked,
e.g. if we want to omit the missing issue mentioned above, we should use
`-Xdoclint:-missing` insteand of `-Xdoclint:none`. The details are
[here](http://docs.oracle.com/javase/8/docs/technotes/tools/unix/javadoc.html#BEJEFABE).
We have to make policies of Javadoc to choose the groups.
- Yes, add configuration `<failOnError>false</failOnError>` to
maven-javadoc-plugin instead of
`<additionalparam>-Xdoclint:none</additionalparam>`, or combination of them if
we choose check groups.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
