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

Hadoop QA commented on MAPREDUCE-1334:
--------------------------------------

+1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12429081/MAPREDUCE-1334.patch
  against trunk revision 908321.

    +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 warnings.

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

    +1 core tests.  The patch passed core unit tests.

    +1 contrib tests.  The patch passed contrib unit tests.

Test results: 
http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/444/testReport/
Findbugs warnings: 
http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/444/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Checkstyle results: 
http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/444/artifact/trunk/build/test/checkstyle-errors.html
Console output: 
http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/444/console

This message is automatically generated.

> contrib/index - test - TestIndexUpdater fails due to an additional presence 
> of file _SUCCESS in hdfs 
> -----------------------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-1334
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1334
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: contrib/index
>            Reporter: Kay Kay
>            Priority: Critical
>             Fix For: 0.21.0
>
>         Attachments: MAPREDUCE-1334.patch
>
>
> $ cd src/contrib/index
> $ ant clean test 
> This fails the test TestIndexUpdater due to a mismatch in the - doneFileNames 
> - data structure, when it is being run with different parameters. 
> (ArrayIndexOutOfBoundsException raised when inserting elements in 
> doneFileNames, array ). 
> Debugging further - there seems to be an additional file called as - 
> hdfs://localhost:36021/myoutput/_SUCCESS , taken into consideration in 
> addition to those that begins with done* .  The presence of the extra file 
> causes the error. 
> Attaching a patch that would circumvent this by increasing the array length 
> of shards by 1 . 
> But longer term the test fixtures need to be probably revisited to see if the 
> presence of _SUCCESS as a file is a good thing to begin with before we even 
> get to this test case. 
> Any comments / suggestions on the same welcome. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to