[GitHub] [accumulo] milleruntime opened a new issue #2299: Explore refactoring ZooCacheFactory

[GitBox <gi...@apache.org>]

milleruntime opened a new issue #2299:
URL: https://github.com/apache/accumulo/issues/2299


   This class is registered as a singleton with SingletonManager. It might be possible to drop this class by moving the getZooCache() methods to their respective places in ServerContext and ClientContext. 
   
   Some of the server code uses this factory to get the systemConfig in `ZooConfigurationFactory` and `ServerConfigurationFactory`, both of which store static objects. The `ServerInfo` object uses this class to get the cache in a few different ways. The different types of `AccumuloConfiguration` objects also use this class to get the property cache.
   
   The old and new client code uses this class to get and store the ZooCache. The static helper class `Tables` uses ZooCacheFactory internally to get ZooCache on demand across different static methods such as exists(), getTableMap() and getTableState().


-- 
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

[GitHub] [accumulo] ctubbsii commented on issue #2299: Explore refactoring ZooCacheFactory

[GitBox <gi...@apache.org>]

ctubbsii commented on issue #2299:
URL: https://github.com/apache/accumulo/issues/2299#issuecomment-934425382


   @EdColeman is working on versioned properties in ZK to reduce the number of watchers per table, among other benefits (I don't have the ticket numbers immediately available to reference). That work is replacing the use of ZooCache with another caching mechanism for config properties stored in ZK. Since config properties are a substantial use case for ZooCache, it might sense to limit the scope of any refactoring here, in order to avoid wasting effort refactoring for uses that are being dropped and to focus on the uses that will remain.


-- 
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