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

Suresh Srinivas commented on HDFS-2907:
---------------------------------------

Comments:
# DFSConfigKeys.java - move the newly added key to the section with "Keys with 
no defaults"
# Some lines in Datanode.java more than 80 columns
# I would have preferred to have a method initStorage(factory) and move the 
code from initBlockPool().
# FSDatasetInterface.java - remove unnecesarry cast to Factory
# SimulatedFSDataset.setStimulatedFSDataset - method namenode could be 
setFactory?
# Since you are changing SimlulatedFSDataset.java - can you please change 
getBlockReport() where after using map, we check for map != null.
# Do not throw IOException from TestSimulatedFSDataset#getSimulatedFSDataset()

                
> Make FSDataset in Datanode Pluggable
> ------------------------------------
>
>                 Key: HDFS-2907
>                 URL: https://issues.apache.org/jira/browse/HDFS-2907
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>            Reporter: Sanjay Radia
>            Assignee: Tsz Wo (Nicholas), SZE
>            Priority: Minor
>         Attachments: h2907_20120216.patch, h2907_20120217.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to