[MAPREDUCE-5342] There are methods that should be deprecated and new methods should be added with names matching purposes

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 2.1.0-beta, 2.0.5-alpha
• Component/s: jobtracker

Environment

Does not matter

• Assignee: Unassigned
• Reporter: Pranay Varma
• Created: 22/Jun/13 22:18
• Updated: 23/Jun/13 06:00

Description

ClusterStatus class has the following methods that need to be deprecated and new methods added:
getMapTasks does not return map tasks, it returns the number of map tasks.

• getBlacklistedTrackers -> getNumBlacklistedTrackers
• getMapTasks -> getNumMapTasks
• getReduceTasks -> getNumReduceTasks
• getTaskTrackers -> getNumTaskTrackers

Cluster class needs the following change:
There is a ClusterStatus class. When getClusterStatus is called, one would expect ClusterStatus to be returned. Instead, one gets ClusterMetrics.

• getClusterStatus -> getClusterMetrics

Job class has the following methods that need to be deprecated and new methods added to match the purposes:

mapProgress suggests that the method is going to map progress and is misleading because, in fact, the method provides progress information about map tasks. It should be deprecated and a method should be added with a name that matches the purpose: getMapTasksProgress or getMapProgress.

• mapProgress -> getMapProgress
• reduceProgress -> getReduceProgress
• cleanupProgress -> getCleanupProgress

JobStatus:

• getQueue -> getQueueName

JobClient:

• getAllJobs -> getJobStatuses

Issue Links

• is duplicated by: MAPREDUCE-5343 reduceProgress is misleading. It should be renamed getReduceTasksProgress
• is duplicated by: MAPREDUCE-5344 getClusterStatus method in Cluster returns ClusterMetrics. It should be called getClusterMetrics or it should return ClusterStatus.
• is duplicated by: MAPREDUCE-5345 cleanupProgress is misleading. It should be renamed getCleanupTasksProgress
• is duplicated by: MAPREDUCE-5347 getBlacklistedTrackers() should be deprecated and a new method called getNumBlacklistedTrackers should be added.