Hadoop Map/Reduce
  1. Hadoop Map/Reduce
  2. MAPREDUCE-5342

There are methods that should be deprecated and new methods should be added with names matching purposes

    Details

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

      Does not matter

      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

          Activity

          Pranay Varma made changes -
          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
          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
          Pranay Varma made changes -
          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
          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
          Pranay Varma made changes -
          Description Cluster 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

          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
          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
          Pranay Varma made changes -
          Link This issue is duplicated by MAPREDUCE-5344 [ MAPREDUCE-5344 ]
          Pranay Varma made changes -
          Link This issue is duplicated by MAPREDUCE-5347 [ MAPREDUCE-5347 ]
          Pranay Varma made changes -
          Description Cluster 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

          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
          *getReduceProgress -> getReduceProgress
          *getCleanupProgress -> getCleanupProgress
          Cluster 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

          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
          Pranay Varma made changes -
          Description

          -Job.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.

          The same applies to the following methods in the Job class:

          getReduceProgress, getCleanupProgress.
          Cluster 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

          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
          *getReduceProgress -> getReduceProgress
          *getCleanupProgress -> getCleanupProgress
          Pranay Varma made changes -
          Summary There are methods in the Job class that should be deprecated and new methods should be added with names matching purposes There are methods that should be deprecated and new methods should be added with names matching purposes
          Pranay Varma made changes -
          Description Job.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.

          The same applies to the following methods in the Job class:

          getReduceProgress, getCleanupProgress.


          -Job.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.

          The same applies to the following methods in the Job class:

          getReduceProgress, getCleanupProgress.
          Pranay Varma made changes -
          Summary There are methods in the Job class that should be deprecated and new methods should be added with names matching purpose There are methods in the Job class that should be deprecated and new methods should be added with names matching purposes
          Pranay Varma made changes -
          Summary mapProgress method in the Job class should be deprecated and a new method called getMapProgress should be added There are methods in the Job class that should be deprecated and new methods should be added with names matching purpose
          Pranay Varma made changes -
          Link This issue is duplicated by MAPREDUCE-5345 [ MAPREDUCE-5345 ]
          Pranay Varma made changes -
          Link This issue is duplicated by MAPREDUCE-5343 [ MAPREDUCE-5343 ]
          Pranay Varma made changes -
          Field Original Value New Value
          Summary mapProgress should be renamed getMapTasksProgress mapProgress method in the Job class should be deprecated and a new method called getMapProgress should be added
          Description 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 renamed to match what it does: getMapTasksProgress. Job.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.

          The same applies to the following methods in the Job class:

          getReduceProgress, getCleanupProgress.
          Pranay Varma created issue -

            People

            • Assignee:
              Unassigned
              Reporter:
              Pranay Varma
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:

                Development