[GitHub] zookeeper pull request #303: ZOOKEEPER-2840: Should using `System.nanoTime()...

2018-04-15 Thread asdf2014
Github user asdf2014 commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/303#discussion_r181613274 --- Diff: src/java/main/org/apache/zookeeper/client/StaticHostProvider.java --- @@ -71,7 +71,7 @@ * if serverAddresses is empty or

[GitHub] zookeeper pull request #303: ZOOKEEPER-2840: Should using `System.nanoTime()...

2017-08-29 Thread maoling
Github user maoling commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/303#discussion_r135789082 --- Diff: src/java/main/org/apache/zookeeper/client/StaticHostProvider.java --- @@ -71,7 +71,7 @@ * if serverAddresses is empty or r

[GitHub] zookeeper pull request #303: ZOOKEEPER-2840: Should using `System.nanoTime()...

2017-07-05 Thread asdf2014
GitHub user asdf2014 opened a pull request: https://github.com/apache/zookeeper/pull/303 ZOOKEEPER-2840: Should using `System.nanoTime() ^ this.hashCode()` fo… Should using `System.nanoTime() ^ this.hashCode()` for StaticHostProvider instead of `System.currentTimeMillis()`. Becaus