[ 
https://issues.apache.org/jira/browse/HDFS-1489?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ivan Kelly updated HDFS-1489:
-----------------------------

    Attachment: NNStorage.diff

I just uploaded a patch which has just the NNStorage changes. Diffed against 
0.22. I can make another against trunk later. I haven't even looked at tests on 
this one. They wont compile now, but should run without issue once reference to 
FSImage are changed to NNStorage. 

This patch come in at 3196 lines. It can't possible be broken any smaller than 
this, because once you stop FSImage from deriving from Storage all hell breaks 
loose.

> breaking the dependency between FSEditLog and FSImage
> -----------------------------------------------------
>
>                 Key: HDFS-1489
>                 URL: https://issues.apache.org/jira/browse/HDFS-1489
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: name-node
>    Affects Versions: 0.21.0
>            Reporter: Diego Marron
>         Attachments: HDFS-1489.diff, HDFS-1489.diff, HDFS-1489.pdf, 
> NNStorage.diff
>
>
> This is a refactor patch which its main concerns are:
> - breaking the dependency between FSEditLog and FSImage
> - Splitting the abstracting the error handling and directory management, 
> - Decoupling Storage from FSImage.
> In order to accomplish the above goal, we will need to introduce new classes:
> -  NNStorage: Will care about the storage. It extends Storage class, and will 
> contain the StorageDirectories.
> -  NNUtils: Some utility static methods on FSImage and FSEditLog will be 
> moved here.
> -  PersistenceManager: FSNameSystem will now be responsible for managing the 
> FSImage & FSEditLog objects. There will be some logic that will have to moved 
> out of FSImage to facilite this. For this we propose a PersistanceManager? 
> object as follows.
> For more deep details, see the design document uploaded.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to