You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mrunit.apache.org by "E. Sammer (JIRA)" <ji...@apache.org> on 2011/05/12 07:05:47 UTC

[jira] [Commented] (MRUNIT-8) Fix failing testConfiguration tests

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

E. Sammer commented on MRUNIT-8:
--------------------------------

I have a hard time seeing how this ever worked. Per the patch on MAPREDUCE-954, I backed out the changes but the original code for MockMapContext has this:

{code}
    public MockMapContext(final List<Pair<KEYIN, VALUEIN>> in, final Counters counters)
        throws IOException, InterruptedException {

      super(new Configuration(),
        new TaskAttemptID("mrunit-jt", 0, true, 0, 0),
        null, null, new MockOutputCommitter(), new MockReporter(counters), null);
    ...
{code}

To me, it seems like you get a new Configuration object no matter what in the Context object which means the configuration passed in by the user (via MapDriver#withConfiguration()) simply never gets used.

Aaron, thoughts?

> Fix failing testConfiguration tests
> -----------------------------------
>
>                 Key: MRUNIT-8
>                 URL: https://issues.apache.org/jira/browse/MRUNIT-8
>             Project: MRUnit
>          Issue Type: Bug
>    Affects Versions: 0.5.0
>            Reporter: E. Sammer
>            Assignee: E. Sammer
>             Fix For: 0.5.0
>
>
> The testConfiguration() tests from TestMapDriver, TestReduceDriver, and TestMapReduceDriver are failing. Not sure if I screwed up MRUNIT-6 or if these always failed. They should be fixed (or removed if they don't test something meaningful).

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