You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Eli Collins (JIRA)" <ji...@apache.org> on 2011/08/11 23:28:27 UTC

[jira] [Commented] (HADOOP-7357) hadoop.io.compress.TestCodec#main() should exit with non-zero exit code if test failed

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

Eli Collins commented on HADOOP-7357:
-------------------------------------

+1 to Phil's latest patch (w/ my small update).  Here are test-patch results:

{noformat}
+1 overall.  

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 3 new or modified tests.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    +1 findbugs.  The patch does not introduce any new Findbugs (version 1.3.9) warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

    +1 system test framework.  The patch passed system test framework compile.
{noformat}


> hadoop.io.compress.TestCodec#main() should exit with non-zero exit code if test failed
> --------------------------------------------------------------------------------------
>
>                 Key: HADOOP-7357
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7357
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: test
>            Reporter: Philip Zeyliger
>            Assignee: Eli Collins
>            Priority: Trivial
>             Fix For: 0.23.0
>
>         Attachments: HADOOP-7357-v2.patch.txt, HADOOP-7357.patch.txt, hadoop-7357-3.patch
>
>
> It's convenient to run something like
> {noformat}
> HADOOP_CLASSPATH=hadoop-test-0.20.2.jar bin/hadoop org.apache.hadoop.io.compress.TestCodec  -count 3 -codec fo
> {noformat}
> but the error code it returns isn't interesting.
> 1-line patch attached fixes that.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira