[MAPREDUCE-5478] TeraInputFormat unnecessarily defines its own FileSplit subclass

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.1.0-beta
• Fix Version/s: 2.1.1-beta
• Component/s: examples
• Hadoop Flags: Reviewed
• Assignee: Sandy Ryza
• Reporter: Sandy Ryza
• Created: 23/Aug/13 00:14
• Updated: 24/Sep/13 23:33
• Resolved: 23/Aug/13 21:28

Description

TeraInputFormat defines its own TeraFileSplit subclass of FileSplit that adds a locations field, which is already included in FileSplit.

This is causing MR2 TeraSort to fail on MR1, which, for a System.arraycopy, requires splits to be of the FileSplit class. While nobody is promising that everything that runs on MR2 should run on MR1, fixing this would be easy and make it possible to compare MR2 TeraSort performance between MR1 and MR2.

We should just get rid of TeraFileSplit and use FileSplit.