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

Suresh Srinivas commented on HDFS-1754:
---------------------------------------

All comments are minor:
# DFSUtil#getInfoServer() 
#* infor to info, nnAddress to httpAddress. In the else clause can you { } 
after if.
#* "else - use old style" to "Use non-federation style configuration"
#* Remove extra line at the end of the method (bef return)
#* some lines are more than 80 chars
# DFSk.java - remove commented code


> Federation HDFS: :testFsck fails
> --------------------------------
>
>                 Key: HDFS-1754
>                 URL: https://issues.apache.org/jira/browse/HDFS-1754
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: name-node
>            Reporter: Boris Shkolnik
>            Assignee: Boris Shkolnik
>         Attachments: HDFS-1754.patch
>
>
> getting config value for HTTP server is changed to read new (federation)style 
> configuration, but it is missing backward compatibility case.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to