[MAPREDUCE-5531] Binary and source incompatibility in mapreduce.TaskID and mapreduce.TaskAttemptID between branch-1 and branch-2

• Type: Sub-task
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 2.1.1-beta
• Fix Version/s: 2.2.0
• Component/s: mrv1, mrv2
• Target Version/s: 2.2.0
• Assignee: Robert Kanter
• Reporter: Robert Kanter
• Created: 24/Sep/13 17:29
• Updated: 30/Jun/15 07:14
• Resolved: 27/Sep/13 19:10

Description

mapreduce.TaskID in branch-1 has these two constructors:

public TaskID(JobID jobId, boolean isMap, int id)
public TaskID(String jtIdentifier, int jobId, boolean isMap, int id)

In branch-2, mapreduce.TaskID no longer has either of the above two constructors.

Also, mapreduce.TaskAttemptID in branch-1 has this constructor:

public TaskAttemptID(String jtIdentifier, int jobId, boolean isMap, int taskId, int id)

In branch-2, mapreduce.TaskAttemptID no longer his this constructor.

It looks like these constructors were probably removed because the boolean isMap was replaced by an enum, TaskType.

This means that any code that tries to use any of those constructors will not be binary or source compatible (in fact, the missing TaskAttemptID constructor calls one of the missing TaskID constructors).