Re: [I] Remove TODOs [accumulo]

["Amemeda (via GitHub)" <gi...@apache.org>]

Amemeda commented on issue #2699:
URL: https://github.com/apache/accumulo/issues/2699#issuecomment-5320003730

   ```java
   zkLockChecker = memoize(() -> {
         // make this use its own ZooSession and ZooCache, because this is used by the
         // tablet location cache, which is a static singleton reused by multiple clients
         // so, it can't rely on being able to continue to use the same client's ZooCache,
         // 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(),
             ZooUtil.getRoot(getInstanceID())).get();
         return new ZookeeperLockChecker(new ZooCache(zk, Set.of(Constants.ZTSERVERS)));
       });
   ``` 
   Is this TODO from ClientContext.java still relevant? the related issue 2301 has been closed for over a year now, but it also looks like this code block was merged in from 2.1 last month


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org