Uploaded image for project: 'Hadoop HDFS'
  1. Hadoop HDFS
  2. HDFS-3498

Make Replica Removal Policy pluggable and ReplicaPlacementPolicyDefault extensible for reusing code in subclass

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.0, 2.0.0-alpha
    • Fix Version/s: 1.2.0, 2.1.0-beta
    • Component/s: namenode
    • Labels:
      None

      Description

      ReplicaPlacementPolicy is already a pluggable component in Hadoop. However, the Replica Removal Policy is still nested in BlockManager that need to be separated out into a ReplicaPlacementPolicy then can be override later. Also it looks like hadoop unit test lack the testing on replica removal policy, so we add it here.
      On the other hand, as a implementation of ReplicaPlacementPolicy, ReplicaPlacementDefault still show lots of generic for other topology cases like: virtualization, and we want to make code in ReplicaPlacementPolicyDefault can be reused as much as possible so a few of its methods were changed from private to protected.

        Attachments

        1. Hadoop-8471-BlockPlacementDefault-extensible.patch
          6 kB
          Junping Du
        2. HDFS-3498.patch
          14 kB
          Junping Du
        3. HDFS-3498-branch-2.patch
          16 kB
          Junping Du
        4. HDFS-3498-v2.patch
          17 kB
          Junping Du
        5. HDFS-3498-v3.patch
          17 kB
          Junping Du
        6. HDFS-3498-v4.patch
          17 kB
          Junping Du
        7. HDFS-3498-v5.patch
          17 kB
          Junping Du

          Issue Links

            Activity

              People

              • Assignee:
                junping_du Junping Du
                Reporter:
                junping_du Junping Du
              • Votes:
                0 Vote for this issue
                Watchers:
                9 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: