fsx379 commented on issue #9660: URL: https://github.com/apache/dubbo/issues/9660#issuecomment-1048512362
> @[fsx379](https://github.com/fsx379) > > 不需要实现一个properties类，比如说可以用一个新的临时的properties对象，将AbstractRegistry的原有properties的数据读出来写到这个临时的properties中。（就像下面代码将原有`cache`的数据写到新的`properties`一样） > > `MetaCacheManager#CacheRefreshTask#run` 3.0分支的代码 如下 > > https://github.com/apache/dubbo/blob/adea101f93555101a5a0eaec6951c055bfe1ede8/dubbo-registry/dubbo-registry-api/src/main/java/org/apache/dubbo/registry/client/metadata/store/MetaCacheManager.java#L153-L167 同步下我这边根据测试，对这个问题的最新看法： 1.通过加日志观察，如果把这个问题优化了，notify总体处理过程会快很多，但是却引发出一个新问题，即 “notify处理了更多的通知” 2.之前工行那边提出了一个“zk客户端延时订阅方案” 的思路，解决同一服务如果部署多个机器，机器在分批重启过程中，造成了大量重复订阅（/dubbo/**serice/provides 下任何节点变化都会推送）。利用延时订阅的方法，减少了频繁变化下的重复请求，降低了带宽和CPU负载。 3.所以目前的properties中的锁，从效果看也是一种 “延时处理方案”，解决了反而引出新问题。。 以上就是我最近的结论，不知道你怎么看？ @BurningCN -- 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]
