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

["Akira AJISAKA (JIRA)" <ji...@apache.org>]

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

Akira AJISAKA commented on MAPREDUCE-5595:
------------------------------------------

The package of the source is o.a.h.mapreduce.task.reduce. Moved to MAPREDUCE project.

> 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
>            Priority: Trivial
>              Labels: newbie
>         Attachments: 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)