[MAPREDUCE-5595] Typo in MergeManagerImpl.java

• Type: Bug
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 2.2.0, 3.0.0-alpha1
• Fix Version/s: 2.6.0
• Labels: newbie

Environment

all

• Target Version/s: 2.6.0
• Assignee: Akira Ajisaka
• Reporter: Efe Gencer
• Created: 25/Oct/13 18:57
• Updated: 12/May/16 18:24
• Resolved: 14/Aug/14 15:52

Description

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); }