owen200008 commented on issue #8895:
URL: https://github.com/apache/dubbo/issues/8895#issuecomment-927445615


   > ```
   >            ZookeeperRegistry(FailbackRegistry).unsubscribe(URL, 
NotifyListener) line: 329  
   >            ListenerRegistryWrapper.unsubscribe(URL, NotifyListener) line: 
129      
   >            RegistryDirectory<T>(DynamicDirectory<T>).destroy() line: 241   
   >            FailoverClusterInvoker<T>(AbstractClusterInvoker<T>).destroy() 
line: 112        
   >            AbstractCluster$InterceptorInvokerNode<T>.destroy() line: 118   
   >            MockClusterInvoker<T>.destroy() line: 78        
   >            MigrationInvoker<T>.destroy() line: 192 
   >            MigrationInvoker<T>(Invoker<T>).destroyAll() line: 50   
   >            CommuLink.close() line: 44      
   >            CommuLink.main(String[]) line: 66
   > ```
   > 
   > 虽然上面是zk的解除注册过程,nacos应该也是如此,可能MockClusterInvoker.java和DynamicDirectory.java 
并不需要修改。
   
   
后来我又想了下,MockClusterInvoker里面还是要调用Directory的destroy的,Directory对象自己没有管理器管理自己的析构,都是在构造函数的时候赋值给别人管理,所以他本身就应该允许多次调用destroy,给固定的容错层来析构感觉不合适


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

Reply via email to