tasanuma commented on pull request #2854:
URL: https://github.com/apache/hadoop/pull/2854#issuecomment-814082007


   On second thought, the last commit has a problem. Just after restarting 
NameNode, NameNode hasn't received any block reports from any DataNode, so 
NameNode recognizes all DataNodes as zero blocks. Therefore, when restarting 
NameNode while decommissioning a DataNode, the DataNode becomes decommissioned 
imediately before replicating its blocks. Actually 
`TestDecommission#testDecommissionWithNamenodeRestart()` covers this case and 
it fails for 0aa3649.
   
   After all, I think we need to consider if the DataNode has zero capacity or 
not. If the capacity is zero, it means the DataNode has a problem with its 
storage, and we can decommission it safely.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to