Issue Details (XML | Word | Printable)

Key: HADOOP-1251
Type: New Feature New Feature
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Owen O'Malley
Reporter: Owen O'Malley
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Hadoop Common

A method to get the InputSplit from a Mapper

Created: 12/Apr/07 05:54 AM   Updated: 08/Jul/09 04:52 PM
Return to search
Component/s: None
Affects Version/s: 0.12.3
Fix Version/s: 0.13.0

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works get-input-split.patch 2007-04-16 10:32 PM Owen O'Malley 7 kB
Text File Licensed for inclusion in ASF works get-input-split.patch 2007-04-12 07:24 AM Owen O'Malley 3 kB
Issue Links:
Dependants
 

Resolution Date: 16/Apr/07 10:57 PM


 Description  « Hide
I need a way to get the InputSplit from a Mapper. This is effectively a stop gap until we get context objects and can fix this much better. smile I introduce a static method in the TaskTracker named getInputSplit() that returns the InputSplit.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order

Doug Cutting added a comment - 16/Apr/07 07:49 PM
This method is not specific to TaskTracker, i.e., it should work fine with LocalRunner too, right? So there ought to be a better place to put it. JobConf? JobClient?

Owen O'Malley added a comment - 16/Apr/07 10:32 PM
This patch moves the getInputSplit from a static method on the TaskTracker to a non-static method on the Reporter. My intention is that getInputSplit will eventually move to the context method along with the rest of the Reporter methods.

This patch also removes the Reporter subclass in the record io testing that was reimplementing Reporter.NULL.


Owen O'Malley added a comment - 16/Apr/07 10:33 PM
Resubmit patch for automatic qa testing.


Doug Cutting added a comment - 16/Apr/07 10:57 PM
I just committed this. Thanks, Owen!

Hadoop QA added a comment - 17/Apr/07 11:21 AM