Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-3237

DatanodeInfo should have a DatanodeID rather than extend it

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      DatanodeInfo currently extends DatanodeID, the code would be more clear if it had a DatanodeID member instead, as DatanodeInfo is private within the server side and DatanodeID is passed to clients.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        33d 20h 18m 1 Eli Collins 13/May/12 21:05
        Eli Collins made changes -
        Assignee Eli Collins [ eli2 ] Eli Collins [ eli ]
        Eli Collins made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Won't Fix [ 2 ]
        Hide
        Eli Collins added a comment -

        We rely on the current structure - DatanodeDescriptor extends DataNodeInfo extends DatanodeID - in a lot of places. This requires more surgery than is probably worthwhile.

        Show
        Eli Collins added a comment - We rely on the current structure - DatanodeDescriptor extends DataNodeInfo extends DatanodeID - in a lot of places. This requires more surgery than is probably worthwhile.
        Eli Collins created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development