ctubbsii opened a new pull request, #5192: URL: https://github.com/apache/accumulo/pull/5192
* Remove ZooSession in favor of a final ZooKeeper instance in the ClientContext/ServerContext that is lazily constructed, and closed when the context closes * Move ZooSession's connection retry logic to ZooUtil (and improve it to log more connection state change events) * Make ZooReader and ZooReaderWriter classes mere decorators/facades for ZooKeeper to make some ZK operations more convenient; they no longer are responsible for maintaining or establishing a ZooKeeper session or client * Get rid of ZooCacheFactory in favor of a lazily constructed ZooCache instance in the ClientContext, to make it more clear when a ZooCache is being shared or reused * Make ZooCache wrap a ZooKeeper object rather than a ZooReader object, and simplify some of its contructor logic * Move instanceId and instanceName lookup logic into ZooUtili * Refactor ClientInfo(Impl) and ServerInfo to simplify the bootstrap logic for clients/contexts/services and to improve code readability; this also involved improvements to token handling and property validation in the ClientInfo code using Optional -- 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]
