[GitHub] accumulo pull request #133: ACCUMULO-4388: Move location to obtain zoo sessi...

2016-07-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/accumulo/pull/133 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is en

[GitHub] accumulo pull request #133: ACCUMULO-4388: Move location to obtain zoo sessi...

2016-07-22 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/133#discussion_r71926509 --- Diff: fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooCache.java --- @@ -155,20 +162,30 @@ public ZooCache(ZooReader reader, Watcher watcher)

[GitHub] accumulo pull request #133: ACCUMULO-4388: Move location to obtain zoo sessi...

2016-07-22 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/133#discussion_r71926540 --- Diff: fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooCache.java --- @@ -155,20 +162,30 @@ public ZooCache(ZooReader reader, Watcher watcher)

[GitHub] accumulo pull request #133: ACCUMULO-4388: Move location to obtain zoo sessi...

2016-07-22 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/133#discussion_r71918556 --- Diff: fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooCache.java --- @@ -225,6 +223,9 @@ public T retry() { if (childrenCache.

[GitHub] accumulo pull request #133: ACCUMULO-4388: Move location to obtain zoo sessi...

2016-07-22 Thread phrocker
GitHub user phrocker opened a pull request: https://github.com/apache/accumulo/pull/133 ACCUMULO-4388: Move location to obtain zoo session. Change the location of obtaining the zoo session. Tests within ZooCacheTest appeared to cover these changes to ensure it did not break funct