[ 
https://issues.apache.org/jira/browse/HDFS-11514?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15925432#comment-15925432
 ] 

Yiqun Lin commented on HDFS-11514:
----------------------------------

Thanks [~vagarychen] for updating the patch. The patch looks good to me. Hi 
[~arpitagarwal], do you have other comments for this? If not, I will commit 
this at the end of day.

> ChooseRandom can potentially be optimized
> -----------------------------------------
>
>                 Key: HDFS-11514
>                 URL: https://issues.apache.org/jira/browse/HDFS-11514
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: namenode
>            Reporter: Chen Liang
>            Assignee: Chen Liang
>         Attachments: HDFS-11514.001.patch, HDFS-11514.002.patch
>
>
> Based on the offline discussion, one potential improvement to the 
> {{chooseRandomWithStorageType}} added in HDFS-11482 is that, currently given 
> a node, the method iterates all its children to sum up the number of 
> candidate datanodes. Since datanode status change is much less frequent than 
> block placement request. It is more efficient to get rid of this iteration 
> check, by probably maintaining another disk type counter map. This JIRA 
> tracks (but not limited) this optimization.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to