[MAPREDUCE-4840] Delete dead code and deprecate public API related to skipping bad records

• Type: Bug
• Status: Patch Available
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: 2.0.0-alpha
• Labels: BB2015-05-TBR
• Assignee: Unassigned
• Reporter: Mostafa Elhemali
• Created: 02/Dec/12 19:14
• Updated: 06/May/15 03:34

Description

It looks like the decision was made in MAPREDUCE-1932 to remove support for skipping bad records rather than fix it (it doesn't work right now in trunk). If that's the case then we should probably delete all the dead code related to it and deprecate the public API's for it right?

Dead code I'm talking about:
1. Task class: skipping, skipRanges, writeSkipRecs
2. MapTask class: SkippingRecordReader inner class
3. ReduceTask class: SkippingReduceValuesIterator inner class
4. Tests: TestBadRecords

Public API:
1. SkipBadRecords class