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 "Akira AJISAKA (JIRA)" <ji...@apache.org> on 2013/10/26 02:44:31 UTC

[jira] [Assigned] (MAPREDUCE-5595) Typo in MergeManagerImpl.java

     [ https://issues.apache.org/jira/browse/MAPREDUCE-5595?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Akira AJISAKA reassigned MAPREDUCE-5595:
----------------------------------------

    Assignee: Akira AJISAKA

> Typo in MergeManagerImpl.java
> -----------------------------
>
>                 Key: MAPREDUCE-5595
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-5595
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>    Affects Versions: 3.0.0, 2.2.0
>         Environment: all
>            Reporter: Efe Gencer
>            Assignee: Akira AJISAKA
>            Priority: Trivial
>              Labels: newbie
>         Attachments: MAPREDUCE-5595.patch, MergeManagerImpl.java
>
>   Original Estimate: 1m
>  Remaining Estimate: 1m
>
> There's a typo ("Invlaid" which should be "Invalid") in line 199 of MergeManagerImpl.java
> currently:
>     if (this.maxSingleShuffleLimit >= this.mergeThreshold) {
>       throw new RuntimeException("Invlaid configuration: "
>           + "maxSingleShuffleLimit should be less than mergeThreshold"
>           + "maxSingleShuffleLimit: " + this.maxSingleShuffleLimit
>           + "mergeThreshold: " + this.mergeThreshold);
>     }
> should be:
>     if (this.maxSingleShuffleLimit >= this.mergeThreshold) {
>       throw new RuntimeException("Invalid configuration: "
>           + "maxSingleShuffleLimit should be less than mergeThreshold"
>           + "maxSingleShuffleLimit: " + this.maxSingleShuffleLimit
>           + "mergeThreshold: " + this.mergeThreshold);
>     }



--
This message was sent by Atlassian JIRA
(v6.1#6144)