You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hbase.apache.org by "Appy (JIRA)" <ji...@apache.org> on 2015/11/27 01:10:11 UTC
[jira] [Created] (HBASE-14890) Fix javadoc checkstyle errors
Appy created HBASE-14890:
----------------------------
Summary: Fix javadoc checkstyle errors
Key: HBASE-14890
URL: https://issues.apache.org/jira/browse/HBASE-14890
Project: HBase
Issue Type: Bug
Reporter: Appy
Assignee: Appy
There are two checkstyles for javadoc: NonEmptyAtclauseDescriptionCheck and JavadocTagContinuationIndentation.
Both require parsing javadoc, but there are some javadocs in codebase which it is not able to parse. Depending on which rule the tool choses to check first it can throw different errors. Now if trunk checks for A first and patch checks for B first, there will be A errors in trunkCheckstyle.xml and B errors in patchCheckstyle.xml. Since B errors are only in patch, checkstyle_report.py reports them are new errors.
Here are two builds proving the point:
https://builds.apache.org/job/PreCommit-HBASE-Build/16668//consoleFull
https://builds.apache.org/job/PreCommit-HBASE-Build/16676/consoleFull
Snippets:
{noformat}
Build 16668:
hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java NonEmptyAtclauseDescriptionCheck 0 1
hbase-common/src/main/java/org/apache/hadoop/hbase/Tag.java NonEmptyAtclauseDescriptionCheck 10 11
hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/RpcClientImpl.java NonEmptyAtclauseDescriptionCheck 2 3
.....
Build 16676:
hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java JavadocTagContinuationIndentationCheck 0 1
hbase-common/src/main/java/org/apache/hadoop/hbase/Tag.java JavadocTagContinuationIndentationCheck 0 1
hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/RequestConverter.java JavadocTagContinuationIndentationCheck 0 1
....
{noformat}
The easiest fix here is to fix these 25 or so javadocs.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)