milleruntime opened a new pull request #1812: URL: https://github.com/apache/accumulo/pull/1812
* Passing a watcher to calls to zk exist method in ZooCache.get() was resulting in watchers hanging around forever after a table is deleted This is a fix for #1809. I am not 100% sure that we don't need a watcher here. The calls to `zooKeeper.exists(zPath, watcher)` happen quite often as this is in the `ZooCache.get()` method but Accumulo doesn't seem to mind with my initial testing. I also removed a very old comment about the importance of having a watcher on this method since I do not think it applies anymore. The node is always added to the cache, whether it has data or not. If the comment does still apply, then it seems like Accumulo is going to retry on a NONODE either way. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
