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

Ming Ma commented on HDFS-9430:
-------------------------------

Thanks [~brahmareddy]. Overall looks good. In addition, should we remove the 
tracking of {{FSNamesystem#imageLoaded}} altogether? [~wheat9] could clarify 
the intention of checking imageLoaded during RPC processing; it seems 
unnecessary given RPC server won't start up until fsimage has been loaded. 
[~vinayrpet] and [~szetszwo] might be able to confirm that {{checkNNStartup}} 
should be enough.

> waitforloadingfsimage() can be removed since checknnstartup() already ensures 
> image loaded and namenode started.
> ----------------------------------------------------------------------------------------------------------------
>
>                 Key: HDFS-9430
>                 URL: https://issues.apache.org/jira/browse/HDFS-9430
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: Brahma Reddy Battula
>            Assignee: Brahma Reddy Battula
>         Attachments: HDFS-9430.patch
>
>
> After initial startup,loading fsimage in between happens only for 
> secondarynn..But it wn't server any client requests.
> So IMO it would be ok to remove it.
> Please check commement [here 
> |https://issues.apache.org/jira/browse/HDFS-9413?focusedCommentId=15005129&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15005129]



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

Reply via email to