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

Akira AJISAKA commented on MAPREDUCE-6595:
------------------------------------------

Ran test-patch.sh for MAPREDUCE-6595.testing.patch with YETUS-271.03.patch.
{noformat}
| Vote |      Subsystem |  Runtime   | Comment
============================================================================
|  +1  |       @author  |  0m 00s    | The patch does not contain any @author 
|      |                |            | tags.
|  -1  |    test4tests  |  0m 00s    | The patch doesn't appear to include any 
|      |                |            | new or modified tests. Please justify why
|      |                |            | no new tests are needed for this patch.
|      |                |            | Also please list what manual steps were
|      |                |            | performed to verify this patch.
|   0  |        mvndep  |  1m 41s    | Maven dependency ordering for branch 
|  +1  |    mvninstall  |  7m 38s    | trunk passed 
|  -1  |       compile  |  2m 17s    | hadoop-mapreduce-project in trunk 
|      |                |            | failed.
|  +1  |    checkstyle  |  0m 22s    | trunk passed 
|  +1  |       mvnsite  |  3m 31s    | trunk passed 
|  +1  |    mvneclipse  |  1m 32s    | trunk passed 
|   0  |      findbugs  |  0m 00s    | Skipped branch modules with no Java 
|      |                |            | source: hadoop-mapreduce-project
|   0  |      findbugs  |  1m 38s    | 
|      |                |            | 
hadoop-mapreduce-project/hadoop-mapreduce-
|      |                |            | client/hadoop-mapreduce-client-core in
|      |                |            | trunk has 2 extant Findbugs warnings.
|  +1  |       javadoc  |  2m 31s    | trunk passed 
|   0  |        mvndep  |  0m 41s    | Maven dependency ordering for patch 
|  +1  |    mvninstall  |  2m 25s    | the patch passed 
|  -1  |       compile  |  1m 37s    | hadoop-mapreduce-project in the patch 
|      |                |            | failed.
|  -1  |         javac  |  1m 37s    | hadoop-mapreduce-project in the patch 
|      |                |            | failed.
|  +1  |    checkstyle  |  0m 25s    | the patch passed 
|  +1  |       mvnsite  |  2m 54s    | the patch passed 
|  +1  |    mvneclipse  |  1m 04s    | the patch passed 
|  +1  |    whitespace  |  0m 00s    | Patch has no whitespace issues. 
|  +1  |           xml  |  0m 01s    | The patch has no ill-formed XML file. 
|   0  |      findbugs  |  0m 00s    | Skipped patch modules with no Java 
|      |                |            | source: hadoop-mapreduce-project
|  +1  |      findbugs  |  1m 32s    | the patch passed 
|  +1  |       javadoc  |  2m 05s    | the patch passed 
|  -1  |    asflicense  |  0m 38s    | Patch generated 9 ASF License warnings. 
|      |                |  36m 13s   | 
{noformat}
The output shows that the 2 findbugs warnings are fixed. (actually ignored)

> Fix findbugs warnings in OutputCommitter and FileOutputCommitter
> ----------------------------------------------------------------
>
>                 Key: MAPREDUCE-6595
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6595
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>            Reporter: Akira AJISAKA
>            Assignee: Akira AJISAKA
>         Attachments: MAPREDUCE-6595.01.patch, MAPREDUCE-6595.testing.patch, 
> findbugsHtml.html
>
>
> There are 2 findbugs warnings in hadoop-mapreduce-client-core module. 
> https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/6237/artifact/patchprocess/branch-findbugs-hadoop-mapreduce-project_hadoop-mapreduce-client_hadoop-mapreduce-client-core-warnings.html



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

Reply via email to