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

Suresh Srinivas commented on HDFS-1986:
---------------------------------------

Comments:
# TestDFSUtil.java - {{InetSocketAddress is = new InetSocketAddress(1234);}} I 
am not clear how this maps to namenode address?
# DFSUtil.java - {{checkSecurity}} could be named {{httpsAddress}}. @param 
checkSecurity needs to be reworded. The method returns an address and not port. 
@return needs to be reworded too.


> Add an option for user to return http or https ports regardless of security 
> is on/off in DFSUtil.getInfoServer()
> ----------------------------------------------------------------------------------------------------------------
>
>                 Key: HDFS-1986
>                 URL: https://issues.apache.org/jira/browse/HDFS-1986
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: tools
>    Affects Versions: 0.23.0
>            Reporter: Tanping Wang
>            Assignee: Tanping Wang
>            Priority: Minor
>             Fix For: 0.23.0
>
>         Attachments: HDFS-1986.patch
>
>
> Currently DFSUtil.getInfoServer gets http port with security off and httpS 
> port with security on.  However, we want to return http port regardless of 
> security on/off for Cluster UI to use.  Add in a third Boolean parameter for 
> user to decide whether to check security or not.

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

Reply via email to