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

Todd Lipcon commented on HDFS-2223:
-----------------------------------

bq. Should FSImage.namesystem be removed?
Yes - it is removed in this patch. I wasn't able to easily remove it from 
BackupImage at this point, without changing a bunch of the BN around. So, I 
left it in the subclass only.

bq. Why not container for FSDirectory, FSImage, FSEditlog, as they are very 
closely related to namespace.
bq. Should NNStorage belong to NameNode?

I think FSEditLog still belongs below FSImage, as it's part of the "image on 
disk" in a sense, and their lifecycle is tightly related.
Moving NNStorage up into NameNode does make a certain amount of sense to me, 
but I didn't want to bite off too much in one patch.


Do you think the changes in the attached patch are good, even if they're not a 
complete cleanup of everything? If so, let's commit this and then do some more 
work in another JIRA, just so each small improvement is easy to review?

> Untangle depencencies between NN components
> -------------------------------------------
>
>                 Key: HDFS-2223
>                 URL: https://issues.apache.org/jira/browse/HDFS-2223
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: name-node
>    Affects Versions: 0.23.0
>            Reporter: Todd Lipcon
>            Assignee: Todd Lipcon
>         Attachments: hdfs-2223-1.txt, hdfs-2223-2.txt, hdfs-2223-3.txt, 
> hdfs-2223-4.txt
>
>
> Working in the NN a lot for HA (HDFS-1623) I've come across a number of 
> situations where the tangled dependencies between NN components has been 
> problematic for adding new features and for testability. It would be good to 
> untangle some of these and clarify what the distinction is between the 
> different components: NameNode, FSNamesystem, FSDirectory, FSImage, 
> NNStorage, and FSEditLog

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to