dlmarion commented on PR #5728: URL: https://github.com/apache/accumulo/pull/5728#issuecomment-3052254412
I was wondering if we might be able to use a ZooCache instance here, instead of doing our own caching. ZooCache has been around for a long time and seems pretty stable. So, I went looking to see if there was one in the Manager that could just be passed down to ZooStore, but I didn't find an instance of it there. However, I did find an instance in ServerInfo. The odd thing though is that the ZooCache instance in ServerInfo appears to be used in the ServerInfo constructor and no where else. It's private and there is no method for accessing it. Do we really have a private, unused ZooCache instance in each server process? I think I need another set of eyes on this, or more coffee. If we wanted to make the ServerInfo ZooCache instance available, then we could easily add methods on ServerInfo and ServerContext. -- 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-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
