Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-3417

Rename BalancerDatanode#getName to getDisplayName to be consistent with Datanode

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.0-alpha
    • Fix Version/s: 2.0.2-alpha
    • Component/s: datanode
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Let's rename BalancerDatanode#getName to getDisplayName to be consistent with Datanode, since it's used for logging. Also makes the code less confusing as "getName" in the context of DataNode/Node usually refers to a network location, not a display name.

      1. hdfs-3417.txt
        4 kB
        Eli Collins

        Activity

        Eli Collins created issue -
        Eli Collins made changes -
        Field Original Value New Value
        Attachment hdfs-3417.txt [ 12526690 ]
        Eli Collins made changes -
        Description Let's rename BalancerDatanode#getName to getDisplayName to be consistent with Datanode, since it's used for logging. Also makes the code less confusing as "getName" in the context of DataNode/Node refers to a network location. Let's rename BalancerDatanode#getName to getDisplayName to be consistent with Datanode, since it's used for logging. Also makes the code less confusing as "getName" in the context of DataNode/Node usually refers to a network location, not a display name.
        Eli Collins made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Eli Collins made changes -
        Priority Major [ 3 ] Minor [ 4 ]
        Eli Collins made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Hadoop Flags Reviewed [ 10343 ]
        Target Version/s 2.0.0 [ 12320353 ]
        Fix Version/s 2.0.0 [ 12320353 ]
        Resolution Fixed [ 1 ]
        Todd Lipcon made changes -
        Fix Version/s 2.0.1 [ 12321440 ]
        Fix Version/s 2.0.0 [ 12320353 ]
        Arun C Murthy made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Eli Collins made changes -
        Assignee Eli Collins [ eli2 ] Eli Collins [ eli ]
        Arun C Murthy made changes -
        Fix Version/s 2.0.2-alpha [ 12322472 ]
        Fix Version/s 2.1.0-alpha [ 12321440 ]

          People

          • Assignee:
            Eli Collins
            Reporter:
            Eli Collins
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development