[MAPREDUCE-4974] Optimising the LineRecordReader initialize() method

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.2-alpha
• Fix Version/s: 2.1.0-beta
• Component/s: mrv1, mrv2, performance
• Labels: patch performance

Environment

Hadoop Linux

• Target Version/s: 1.0.0, 1.0.4, 1.1.1, 2.0.0-alpha, 2.0.1-alpha, 0.23.4, 0.23.5
• Assignee: Gelesh
• Reporter: Arun A K
• Due: 04/Feb/13
• Created: 04/Feb/13 12:25
• Updated: 10/Mar/15 04:30
• Resolved: 15/Apr/13 21:37

Description

I found there is a a scope of optimizing the code, over initialize() if we have compressionCodecs & codec instantiated only if its a compressed input.
Mean while Gelesh George Omathil, added if we could avoid the null check of key & value. This would time save, since for every next key value generation, null check is done. The intention being to instantiate only once and avoid NPE as well. Hope both could be met if initialize key & value over initialize() method. We both have worked on it.