Github user lvfangmin commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/534#discussion_r192627337
--- Diff: src/java/main/org/apache/zookeeper/client/StaticHostProvider.java
---
@@ -73,15 +80,27 @@
* if serverAddresses is empty or resolves to an empty list
*/
public StaticHostProvider(Collection<InetSocketAddress>
serverAddresses) {
- sourceOfRandomness = new Random(System.currentTimeMillis() ^
this.hashCode());
+ init(serverAddresses,
--- End diff --
Nit: Call StaticHostProvider(Collection<InetSocketAddress> serverAddresses,
Resolver resolver).
Please change the other two constructor as well.
---
