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

Arpit Agarwal commented on HDFS-7411:
-------------------------------------

bq. So what's broken right now is that these UC blocks aren't picked up by 
decom, since decom scans the DN block list. This means these DNs can be 
erroneously decommissioned. 

That makes sense. Yes logically it makes sense to include UC blocks in the DN 
block list. Perhaps it's safe to do so in addStoredBlockUnderConstruction if we 
can make check it doesn't break any assumption elsewhere in BlockManager.

> Refactor and improve decommissioning logic into DecommissionManager
> -------------------------------------------------------------------
>
>                 Key: HDFS-7411
>                 URL: https://issues.apache.org/jira/browse/HDFS-7411
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>    Affects Versions: 2.5.1
>            Reporter: Andrew Wang
>            Assignee: Andrew Wang
>         Attachments: hdfs-7411.001.patch, hdfs-7411.002.patch, 
> hdfs-7411.003.patch, hdfs-7411.004.patch
>
>
> Would be nice to split out decommission logic from DatanodeManager to 
> DecommissionManager.



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

Reply via email to