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

Hairong Kuang commented on HDFS-1473:
-------------------------------------

Sounds good to me. Personally I feel that it is simpler without the inner 
classes. But it is your call.

> Refactor storage management into separate classes than fsimage file 
> reading/writing
> -----------------------------------------------------------------------------------
>
>                 Key: HDFS-1473
>                 URL: https://issues.apache.org/jira/browse/HDFS-1473
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Todd Lipcon
>            Assignee: Todd Lipcon
>         Attachments: hdfs-1473-prelim.txt
>
>
> Currently the FSImage class is responsible both for storage management (eg 
> moving around files, tracking file names, the VERSION file, etc) as well as 
> for the actual serialization and deserialization of the "fsimage" file within 
> the storage directory.
> I'd like to refactor the loading and saving code into new classes. This will 
> make testing easier and also make the major changes in HDFS-1073 easier to 
> understand.

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