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

Jing Zhao commented on HDFS-8005:
---------------------------------

bq. Other than that, the 001 patch looks good

Thanks for the review, Zhe! Can I get your +1 for the latest patch?

> Erasure Coding: simplify striped block recovery work computation and add tests
> ------------------------------------------------------------------------------
>
>                 Key: HDFS-8005
>                 URL: https://issues.apache.org/jira/browse/HDFS-8005
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Jing Zhao
>            Assignee: Jing Zhao
>         Attachments: HDFS-8005.000.patch, HDFS-8005.000.patch, 
> HDFS-8005.001.patch
>
>
> HDFS-7369 adds the functionality to distribute recovery work of striped 
> blocks to datanodes. There are still some pending issues:
> # In {{BlockManager#chooseSourceNode}}, a node is added into 
> {{healthyIndices}} without checking if its block is live and healthy
> # The test {{TestRecoverStripedBlcoks#testMissingStripedBlock}} has not 
> tested striped blocks because the file is created before setting the storage 
> policy
> # In {{computeRecoveryWorkForBlocks}}, instead of using 
> {{BlockCollection#isStriped}}, we'd better use {{BlockInfo#isStriped}}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to