[GitHub] [dubbo] pinxiong commented on pull request #9015: [3.0] change getRegistries return type to HashSet for unique, close zookeeper client that is no long used by any application

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

pinxiong commented on pull request #9015:
URL: https://github.com/apache/dubbo/pull/9015#issuecomment-943161362


   For the specified `Application`, there are several compoments, such as `Metada`, `Registry` etc.
   
   Some complex scenarios will take place in the future, if zk client supoorts to be shared between different `Applications` below
   
   + There are `Application1#Metada` and `Application2#Registry` in `zkClient1`
   + There are `Application2#Metada` and `Application1#Registry` in `zkClient2`
   
   The relationship between `zkClient` and `Application` are very complex, also the purpose of sharing `zkClient` is not achieved if not configured correctly.
   
   In my opinion, it's necessary to support shared `zkClient` betweent different components in the same `Application`.


-- 
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@dubbo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org