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

Mingliang Liu commented on HDFS-9129:
-------------------------------------

Thanks to [~anu] for the review and comment.

Anu and I are discussing the potential conflicts with [HDFS-4015] offline, and 
I'll update the patch soon once we reach consensus.

{quote}
 I think it might be a good idea to move BytesInFutureBlocks also to the same 
class.
{quote}
Agreed. Let's address that separately as this jira is to move safemode block 
count from {{FSNamesystem}} to {{BlockManager}}. I'll report this after this is 
resolved, and invite you to review the work.

> Move the safemode block count into BlockManager
> -----------------------------------------------
>
>                 Key: HDFS-9129
>                 URL: https://issues.apache.org/jira/browse/HDFS-9129
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Haohui Mai
>            Assignee: Mingliang Liu
>         Attachments: HDFS-9129.000.patch, HDFS-9129.001.patch, 
> HDFS-9129.002.patch, HDFS-9129.003.patch, HDFS-9129.004.patch, 
> HDFS-9129.005.patch, HDFS-9129.006.patch, HDFS-9129.007.patch, 
> HDFS-9129.008.patch, HDFS-9129.009.patch, HDFS-9129.010.patch, 
> HDFS-9129.011.patch
>
>
> The {{SafeMode}} needs to track whether there are enough blocks so that the 
> NN can get out of the safemode. These fields can moved to the 
> {{BlockManager}} class.



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

Reply via email to