[ 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)