You are viewing a plain text version of this content. The canonical link for it is here.
Posted to mapreduce-issues@hadoop.apache.org by "Jothi Padmanabhan (JIRA)" <ji...@apache.org> on 2009/07/22 08:57:14 UTC

[jira] Created: (MAPREDUCE-785) Refactor TestReduceFetchFromPartialMem into a separate test

Refactor TestReduceFetchFromPartialMem into a separate test 
------------------------------------------------------------

                 Key: MAPREDUCE-785
                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-785
             Project: Hadoop Map/Reduce
          Issue Type: Test
            Reporter: Jothi Padmanabhan
            Assignee: Jothi Padmanabhan


The rationale behind doing this is to enable this test alone to be included in the commit-tests target

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


[jira] Updated: (MAPREDUCE-785) Refactor TestReduceFetchFromPartialMem into a separate test

Posted by "Jothi Padmanabhan (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MAPREDUCE-785?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jothi Padmanabhan updated MAPREDUCE-785:
----------------------------------------

    Attachment: mapred-785.patch

Patch that splits the tests. Thanks Konstantin for help with this patch

> Refactor TestReduceFetchFromPartialMem into a separate test 
> ------------------------------------------------------------
>
>                 Key: MAPREDUCE-785
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-785
>             Project: Hadoop Map/Reduce
>          Issue Type: Test
>            Reporter: Jothi Padmanabhan
>            Assignee: Jothi Padmanabhan
>             Fix For: 0.21.0
>
>         Attachments: mapred-785.patch
>
>
> The rationale behind doing this is to enable this test alone to be included in the commit-tests target

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


[jira] Updated: (MAPREDUCE-785) Refactor TestReduceFetchFromPartialMem into a separate test

Posted by "Jothi Padmanabhan (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MAPREDUCE-785?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jothi Padmanabhan updated MAPREDUCE-785:
----------------------------------------

    Release Note: Moves TestReduceFetchFromPartialMem out of TestReduceFetch into a separate test to enable it to be included in the commit-tests target.

> Refactor TestReduceFetchFromPartialMem into a separate test 
> ------------------------------------------------------------
>
>                 Key: MAPREDUCE-785
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-785
>             Project: Hadoop Map/Reduce
>          Issue Type: Test
>            Reporter: Jothi Padmanabhan
>            Assignee: Jothi Padmanabhan
>             Fix For: 0.21.0
>
>         Attachments: mapred-785.patch
>
>
> The rationale behind doing this is to enable this test alone to be included in the commit-tests target

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


[jira] Updated: (MAPREDUCE-785) Refactor TestReduceFetchFromPartialMem into a separate test

Posted by "Chris Douglas (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MAPREDUCE-785?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Chris Douglas updated MAPREDUCE-785:
------------------------------------

      Resolution: Fixed
    Hadoop Flags: [Reviewed]
          Status: Resolved  (was: Patch Available)

+1 Looks good. The refactored tests passed with the expected semantics.
{noformat}
     [exec] +1 overall.  
     [exec] 
     [exec]     +1 @author.  The patch does not contain any @author tags.
     [exec] 
     [exec]     +1 tests included.  The patch appears to include 6 new or modified tests.
     [exec] 
     [exec]     +1 javadoc.  The javadoc tool did not generate any warning messages.
     [exec] 
     [exec]     +1 javac.  The applied patch does not increase the total number of javac compiler warnings.
     [exec] 
     [exec]     +1 findbugs.  The patch does not introduce any new Findbugs warnings.
     [exec] 
     [exec]     +1 release audit.  The applied patch does not increase the total number of release audit warnings.
{noformat}

I committed this. Thanks, Jothi!

> Refactor TestReduceFetchFromPartialMem into a separate test 
> ------------------------------------------------------------
>
>                 Key: MAPREDUCE-785
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-785
>             Project: Hadoop Map/Reduce
>          Issue Type: Test
>            Reporter: Jothi Padmanabhan
>            Assignee: Jothi Padmanabhan
>             Fix For: 0.21.0
>
>         Attachments: mapred-785.patch
>
>
> The rationale behind doing this is to enable this test alone to be included in the commit-tests target

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


[jira] Updated: (MAPREDUCE-785) Refactor TestReduceFetchFromPartialMem into a separate test

Posted by "Jothi Padmanabhan (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MAPREDUCE-785?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jothi Padmanabhan updated MAPREDUCE-785:
----------------------------------------

    Fix Version/s: 0.21.0
           Status: Patch Available  (was: Open)

> Refactor TestReduceFetchFromPartialMem into a separate test 
> ------------------------------------------------------------
>
>                 Key: MAPREDUCE-785
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-785
>             Project: Hadoop Map/Reduce
>          Issue Type: Test
>            Reporter: Jothi Padmanabhan
>            Assignee: Jothi Padmanabhan
>             Fix For: 0.21.0
>
>         Attachments: mapred-785.patch
>
>
> The rationale behind doing this is to enable this test alone to be included in the commit-tests target

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