Haohui Mai created HADOOP-11387:
-----------------------------------

             Summary: Simplify NetUtils#canonicalizeHost()
                 Key: HADOOP-11387
                 URL: https://issues.apache.org/jira/browse/HADOOP-11387
             Project: Hadoop Common
          Issue Type: Bug
            Reporter: Haohui Mai
            Assignee: Haohui Mai


Currently {{NetUtils#canonicalizeHost}} uses a {{ConcurrentHashMap}} to cache 
the canonicalized hostname.

{code}
  private static String canonicalizeHost(String host) {
    // check if the host has already been canonicalized
    String fqHost = canonicalizedHostCache.get(host);
    if (fqHost == null) {
      try {
        fqHost = SecurityUtil.getByName(host).getHostName();
        // slight race condition, but won't hurt
        canonicalizedHostCache.put(host, fqHost);
      } catch (UnknownHostException e) {
        fqHost = host;
      }
    }
    return fqHost;
  }
{code}

The code can be simplified using {{CacheMap}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to