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

Andrew Wang commented on HDFS-7411:
-----------------------------------

Hey Arpit,

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. I'm not sure how to resolve this without adding the UC blocks 
to the DN block list. Good point about that second if statement too.

I guess I'll dig more on this myself; if anyone else knows about this, a 
comment would be appreciated. At a logical level, I'm not sure why we'd have 
the DN locations in the blocksMap, but not also add the block to the DN's block 
list.

> 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