[ https://issues.apache.org/jira/browse/ZOOKEEPER-2171?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14504028#comment-14504028 ]
Michi Mutsuzaki commented on ZOOKEEPER-2171: -------------------------------------------- Thanks Raul. We should replace getHostName() with getHostString(), and also remove src/java/main/org/apache/zookeeper/common/HostNameUtils.java. I don't think the code relies on getHostName() performing a reverse dns lookup, so replacing it with getHostString() shouldn't cause any correctness issues. > avoid reverse lookups in QuorumCnxManager > ----------------------------------------- > > Key: ZOOKEEPER-2171 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2171 > Project: ZooKeeper > Issue Type: Bug > Components: quorum > Reporter: Raul Gutierrez Segales > Assignee: Raul Gutierrez Segales > Fix For: 3.5.1, 3.6.0 > > > Apparently, ZOOKEEPER-107 (via a quick git-blame look) introduced a bunch of > getHostName() calls in QCM. Besides the overhead, these can cause problems > when mixed with failing/mis-configured DNS servers. > It would be nice to reduce them, if that doesn't affect operational > correctness. -- This message was sent by Atlassian JIRA (v6.3.4#6332)