dlmarion commented on code in PR #5256:
URL: https://github.com/apache/accumulo/pull/5256#discussion_r1915351169
##########
core/src/main/java/org/apache/accumulo/core/clientImpl/ClientContext.java:
##########
@@ -1061,8 +1062,9 @@ public synchronized ZookeeperLockChecker
getTServerLockChecker() {
// because that client could be closed, and its ZooSession also closed
// this needs to be fixed; TODO
https://github.com/apache/accumulo/issues/2301
var zk =
info.getZooKeeperSupplier(ZookeeperLockChecker.class.getSimpleName()).get();
- this.zkLockChecker =
- new ZookeeperLockChecker(new ZooCache(zk), getZooKeeperRoot(),
getServerPaths());
+ this.zkLockChecker = new ZookeeperLockChecker(
+ new ZooCache(zk, Optional.empty(),
ZooUtil.getRoot(getInstanceID())), getZooKeeperRoot(),
Review Comment:
Fixed in 0d0a9cd
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
