[ 
http://issues.apache.org/jira/browse/HADOOP-335?page=comments#action_12421038 ] 

Doug Cutting commented on HADOOP-335:
-------------------------------------

You mix indentation and style changes with functional changes.  Can you please 
keep these separate?  If you like, make the style changes, but as a separate 
patch, so that your patch is easier to evaluate.  It's a good practice to 
preview your patches and make sure that each change in the patch is required by 
the issue the patch purports to fix.


> factor out the namespace image/transaction log writing
> ------------------------------------------------------
>
>          Key: HADOOP-335
>          URL: http://issues.apache.org/jira/browse/HADOOP-335
>      Project: Hadoop
>         Type: Improvement

>     Reporter: Owen O'Malley
>     Assignee: Konstantin Shvachko
>  Attachments: FSImage_Refactor1.patch
>
> Factor the checkpoint/transaction log handling code out of FSNameSystem into 
> its own class.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to