[MAPREDUCE-5529] Binary incompatibilities in mapred.lib.TotalOrderPartitioner 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 00:01
• Updated: 30/Jun/15 07:14
• Resolved: 27/Sep/13 20:05

Description

mapred.lib.TotalOrderPartitioner in branch-1 has these two methods:

public static String getPartitionFile(JobConf job)
public static void setPartitionFile(JobConf job, Path p)

In branch-2, mapred.lib.TotalOrderPartitioner is now a subclass of mapred.lib.TotalOrderPartitioner, from which it inherits the similar methods:

public static String getPartitionFile(Configuration conf)
public static void setPartitionFile(Configuration conf, Path p)

This means that any code that does either of the following:

TotalOrderPartitioner.setPartitionFile(new JobConf(), new Path("/"));
String str = TotalOrderPartitioner.getPartitionFile(new JobConf());

will not be binary compatible (that is, if compiled against branch-1, it will throw a NoSuchMethodError if run against branch-2).