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

getBlacklistedTrackers() should be deprecated and a new method called getNumBlacklistedTrackers should be added.

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Duplicate
    • Affects Version/s: 2.1.0-beta, 2.0.5-alpha
    • Fix Version/s: None
    • Labels:
      None

      Description

      If the method returns a number, then that is what the method name should reflect. It should be called getNumBlacklistedTrackers(). The method does not return trackers, it returns number of trackers.

      The same applies to the following:

      • getMapTasks, getReduceTasks, getTaskTrackers

      These should be deprecated and a new method should be added: getNumxxx.

        Issue Links

          Activity

          Pranay Varma created issue -
          Pranay Varma made changes -
          Field Original Value New Value
          Description If the method returns a number, then that is what the method name should reflect. It should be called getNumBlacklistedTrackers(); If the method returns a number, then that is what the method name should reflect. It should be called getNumBlacklistedTrackers();

          The same applies to the following:
          - getMapTasks, getReduceTasks, getTaskTrackers

          These should be deprecated and a new method should be added: getNumxxx.
          Pranay Varma made changes -
          Summary getBlacklistedTrackers() is misleading. It does not return trackers. It should be renamed getNumBlacklistedTrackers. getBlacklistedTrackers() should be deprecated and a new method called getNumBlacklistedTrackers should be added.
          Description If the method returns a number, then that is what the method name should reflect. It should be called getNumBlacklistedTrackers();

          The same applies to the following:
          - getMapTasks, getReduceTasks, getTaskTrackers

          These should be deprecated and a new method should be added: getNumxxx.
          If the method returns a number, then that is what the method name should reflect. It should be called getNumBlacklistedTrackers(). The method does not return trackers, it returns number of trackers.

          The same applies to the following:
          - getMapTasks, getReduceTasks, getTaskTrackers

          These should be deprecated and a new method should be added: getNumxxx.
          Pranay Varma made changes -
          Resolution Duplicate [ 3 ]
          Status Open [ 1 ] Resolved [ 5 ]
          Pranay Varma made changes -
          Link This issue duplicates MAPREDUCE-5342 [ MAPREDUCE-5342 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development