[MAPREDUCE-4990] Construct debug strings conditionally in ShuffleHandler.Shuffle#sendMapOutput()

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 2.0.3-alpha, 0.23.5
• Fix Version/s: 0.23.7, 2.1.0-beta
• Target Version/s: 0.23.7, 2.1.0-beta
• Hadoop Flags: Reviewed
• Assignee: Karthik Kambatla
• Reporter: Karthik Kambatla
• Created: 08/Feb/13 06:28
• Updated: 03/Nov/14 18:33
• Resolved: 18/Mar/13 18:46

Description

In #sendMapOutput(), the debug statements are not wrapped in the customary #isDebugEnabled() condition. Given this piece of code is critical for performance, it would be nice to fix it.