Github user lvfangmin commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/534#discussion_r194215321
--- Diff: src/java/main/org/apache/zookeeper/client/StaticHostProvider.java
---
@@ -314,7 +340,7 @@ public InetSocketAddress next(long spinDelay) {
addr = nextHostInReconfigMode();
if (addr != null) {
currentIndex = serverAddresses.indexOf(addr);
- return addr;
+ return resolve(addr);
--- End diff --
I know there is DNS caching, not aware that we have JVM or os caching for
this, if we tested this is trivial, I totally agree we should keep this simple.
---
