You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "dschneider-pivotal (GitHub)" <gi...@apache.org> on 2019/04/03 16:53:33 UTC

[GitHub] [geode] dschneider-pivotal commented on pull request #3385: GEODE-6580: use ConcurrentHashMap for host names

the old condition for terminating the loop was if lhName != null. lhName is the result of calling reverseDNS which can return null. So in the new code shouldn't this "break;" be in "if (addrName != null)"?

[ Full content available at: https://github.com/apache/geode/pull/3385 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org