dpol1 commented on issue #3033: URL: https://github.com/apache/hugegraph/issues/3033#issuecomment-4497186718
@dosu Where is the per-graph CacheManager singleton instantiated and disposed today? List the call sites that create the per-spaceGraphName cache instances and any hook (graph open/close/dispose) where store listener register/unregister could naturally live if we move them out of listenChanges/unlistenChanges. Does CachedSchemaTransactionV2 have an equivalent storeEventListenStatus-style tracking field for its store listener? If yes, does it share the close of T1+T2 possible? This affects whether the fix needs synchronized or whether ConcurrentHashMap.compute() (like the CacheListenerHolder pattern) is sufficient. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
