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

Jing Zhao commented on HDFS-8909:
---------------------------------

I do not think it's necessary to do another ID check here. Also I think it's 
fine to expose striped to BlockUnderConstructionFeature, since anyway it 
handles striped/contiguous replicas in different ways. Besides, as we have 
discussed, the reported block is added as a replica only if its block id is 
mapped to the BlockInfo object, so the assertion is not that necessary.

> Erasure coding: update BlockInfoContiguousUC and BlockInfoStripedUC to use 
> BlockUnderConstructionFeature
> --------------------------------------------------------------------------------------------------------
>
>                 Key: HDFS-8909
>                 URL: https://issues.apache.org/jira/browse/HDFS-8909
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: namenode
>    Affects Versions: HDFS-7285
>            Reporter: Zhe Zhang
>            Assignee: Jing Zhao
>         Attachments: HDFS-8909.000.patch, HDFS-8909.001.patch, 
> HDFS-8909.002.patch
>
>
> HDFS-8801 converts {{BlockInfoUC}} as a feature. We should consolidate 
> {{BlockInfoContiguousUC}} and {{BlockInfoStripedUC}} logics to use this 
> feature.



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

Reply via email to