ctubbsii edited a comment on issue #1225: Use fewer ZooKeeper watchers URL: https://github.com/apache/accumulo/issues/1225#issuecomment-520001967 The (potential) update to ZK 3.5 is unrelated to this issue. It was mentioned here merely for the possibility of using local session feature, which is one of many possible ways to reduce the number of watchers we use. If we want to use that feature (which is only one possible way to reduce the number of watchers... and not necessarily even the best way to do so), then upgrading is a prerequisite, but otherwise, it is not related to this issue. Please create a separate issue if you want to pursue upgrading ZK to 3.5. @jzgithub1 The build error you were getting is because we have strict dependency checks in the build. That error indicates we need to explicitly name a direct dependency, if we are using code that is actually located in a transitive dependency... or that we should remove a declared dependency if we are not directly using code from that dependency. As for the origin of zookeeper-jute... I don't know what that repo is that you found, but I found the zookeeper-jute code in ZooKeeper's repository here: https://github.com/apache/zookeeper/tree/master/zookeeper-jute quite easily... it's just another component of ZooKeeper's source code, split into a sub-module, as @nkalmar says. @nkalmar If we update to require ZK 3.5, rest assured, we will update to the latest patch version. I'm not sure how we're using curator, but it's very limited, and I'm not sure the code that uses it is current at all.
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services
