[HBASE-19192] HMaster#getRegionServerInfoPort() should respect config value of -1

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Not A Problem
• Assignee: Janos Gub
• Reporter: Ted Yu
• Created: 06/Nov/17 22:04
• Updated: 01/Aug/18 06:20
• Resolved: 04/Dec/17 15:51

Description

  public int getRegionServerInfoPort(final ServerName sn) {
    RegionServerInfo info = this.regionServerTracker.getRegionServerInfo(sn);
    if (info == null || info.getInfoPort() == 0) {
      return conf.getInt(HConstants.REGIONSERVER_INFO_PORT,
        HConstants.DEFAULT_REGIONSERVER_INFOPORT);
    }
    return info.getInfoPort();

hbase.regionserver.info.port config is only checked when regionServerTracker doesn't have info port.
When hbase.regionserver.info.port is set to -1 by user, we should respect the config value and disable UI.