[ 
https://issues.apache.org/jira/browse/HADOOP-12312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14680476#comment-14680476
 ] 

Allen Wittenauer commented on HADOOP-12312:
-------------------------------------------


YARN-3232 generated the following with HADOOP-12111+HADOOP-12129 and unit tests 
turned off:

{code}
| Vote |      Subsystem |  Runtime   | Comment
============================================================================
|  +1  |    mvninstall  |  4m 12s    | trunk passed 
|  +1  |         javac  |  4m 24s    | trunk passed 
|  +1  |       javadoc  |  2m 45s    | trunk passed 
|  +1  |          site  |  2m 04s    | trunk passed 
|  +1  |       @author  |  0m 00s    | The patch does not contain any @author 
|      |                |            | tags.
|  +1  |    test4tests  |  0m 00s    | The patch appears to include 6 new or 
|      |                |            | modified test files.
|  +1  |    checkstyle  |  0m 31s    | trunk passed 
|  +1  |         javac  |  4m 25s    | the patch passed 
|  +1  |          site  |  2m 04s    | the patch passed 
|  +1  |    asflicense  |  0m 16s    | Patch does not generate ASF License 
|      |                |            | warnings.
|  -1  |    checkstyle  |  0m 32s    | Patch generated 1 new checkstyle issues 
|      |                |            | in . (total was 299, now 296).
|  +1  |    whitespace  |  0m 00s    | Patch has no whitespace issues. 
|  +1  |    mvninstall  |  1m 59s    | the patch passed 
|  +1  |       javadoc  |  2m 49s    | the patch passed 
|  +1  |       eclipse  |  1m 19s    | the patch passed 
|  +1  |      findbugs  |  4m 35s    | the patch passed 
|      |                |  36m 51s   | 


|| Subsystem || Report/Notes ||
============================================================================
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12749469/YARN-3232.02.patch |
| JIRA Issue | YARN-3232 |
| git revision | trunk / 8f73bdd |
| Optional Tests | asflicense javac javadoc mvninstall unit findbugs checkstyle 
site |
| uname | Darwin aw-mbp-work.local 13.4.0 Darwin Kernel Version 13.4.0: Wed Mar 
18 16:20:14 PDT 2015; root:xnu-2422.115.14~1/RELEASE_X86_64 x86_64 |
| Build tool | maven |
| Personality | 
/Users/aw/Src/aw-github/hadoop-yetus/dev-support/personality/hadoop.sh |
| Default Java | 1.7.0_67 |
| findbugs | v3.0.0 |
| checkstyle | /Users/aw/Src/drd-github/patchprocess/diff-checkstyle-root.txt |
| Max memory used | 95MB |
{code}

So this has likely already been fixed.

> Findbugs HTML report link shows 0 warnings despite errors
> ---------------------------------------------------------
>
>                 Key: HADOOP-12312
>                 URL: https://issues.apache.org/jira/browse/HADOOP-12312
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: yetus
>            Reporter: Varun Saxena
>
> Refer to Hadoop QA report below :
> https://issues.apache.org/jira/browse/YARN-3232?focusedCommentId=14679146&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14679146
> The report shows -1 for findbugs because there have been 7 findbugs warnings 
> introduced. But the HTML report in link is showing 0 findbugs warnings.
> I verified locally and the warnings did indeed exist.
> So there must be some problem in findbugs HTML report generation in 
> test-patch.sh
> This inconsistency between -1 for findbugs and HTML report lead to these 
> findbugs warnings being leaked to trunk.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to