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

Lei (Eddy) Xu commented on HDFS-9835:
-------------------------------------

Hi, [~cmccabe]

The patch looks good to me overall.

+1 after addressing final nitpicks:

* There are some checkstyle warnings  for {{switch...case}} indentions. 
* The findbugs warning is false, but can we mitigate it somehow?
* Could you also apply XATTR_..._MASKs to {{PBImageXmlWriter#dumpXAttrs}}. 


Thanks a lot for the work.

> OIV: add ReverseXML processor which reconstructs an fsimage from an XML file
> ----------------------------------------------------------------------------
>
>                 Key: HDFS-9835
>                 URL: https://issues.apache.org/jira/browse/HDFS-9835
>             Project: Hadoop HDFS
>          Issue Type: New Feature
>          Components: tools
>    Affects Versions: 2.0.0-alpha
>            Reporter: Colin Patrick McCabe
>            Assignee: Colin Patrick McCabe
>         Attachments: HDFS-9835.001.patch, HDFS-9835.002.patch, 
> HDFS-9835.003.patch, HDFS-9835.004.patch, HDFS-9835.005.patch
>
>
> OIV: add ReverseXML processor which reconstructs an fsimage from an XML file. 
>  This will make it easy to create fsimages for testing, and manually edit 
> fsimages when there is corruption.



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

Reply via email to