[ 
https://issues.apache.org/jira/browse/HDFS-6828?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tsz Wo Nicholas Sze updated HDFS-6828:
--------------------------------------

    Attachment: h6828_20140808c.patch

Vinay, thanks a lot for reviewing the patch!  Here is a new patch incorporated 
all your comments.

h6828_20140808c.patch:
# Fixed.
# Changed all Time.now() to Time.monotonicNow().
# You are right that I should return a trimmed list in Dispatcher#init(..).  
Then, Balancer#init(..) does not have to check shouldIgnore() again.
# Added javadoc.

> Separate block replica dispatching from Balancer
> ------------------------------------------------
>
>                 Key: HDFS-6828
>                 URL: https://issues.apache.org/jira/browse/HDFS-6828
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: balancer
>            Reporter: Tsz Wo Nicholas Sze
>            Assignee: Tsz Wo Nicholas Sze
>         Attachments: h6828_20140808.patch, h6828_20140808b.patch, 
> h6828_20140808c.patch
>
>
> The Balancer class implements two major features, (1) balancing logic for 
> selecting replicas in order to balance the cluster and (2) block replica 
> dispatching for moving the block replica around.  This JIRA is to separate 
> (2) from Balancer so that the code could be reused by other code such as the 
> new data migration tool proposed in HDFS-6801.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to