Arpit Agarwal created HDFS-5272:
-----------------------------------

             Summary: DFSUtil#getInfoServer cleanup
                 Key: HDFS-5272
                 URL: https://issues.apache.org/jira/browse/HDFS-5272
             Project: Hadoop HDFS
          Issue Type: Bug
          Components: namenode
    Affects Versions: 3.0.0
            Reporter: Arpit Agarwal


The {{httpsAddress}} parameter is not used outside of tests.

{code}
  public static String getInfoServer(InetSocketAddress namenodeAddr,
      Configuration conf, boolean httpsAddress) throws IOException {
    boolean securityOn = UserGroupInformation.isSecurityEnabled();
    String httpAddressKey = (securityOn && httpsAddress) ? 
        DFS_NAMENODE_HTTPS_ADDRESS_KEY : DFS_NAMENODE_HTTP_ADDRESS_KEY;
    String httpAddressDefault = (securityOn && httpsAddress) ? 
        DFS_NAMENODE_HTTPS_ADDRESS_DEFAULT : DFS_NAMENODE_HTTP_ADDRESS_DEFAULT;
{code}

It sounds DFS_NAMENODE_HTTP_ADDRESS_DEFAULT is going to be the default that is 
always available so there should be no reason to use 
DFS_NAMENODE_HTTPS_ADDRESS_DEFAULT. Filing Jira to clear this up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to