zrlw commented on issue #9239: URL: https://github.com/apache/dubbo/issues/9239#issuecomment-966958950
搭建了一套nacos测试了一下dubbo-samples集成测试涉及nacos的四个工程：dubbo-samples-nacos-conditionrouter、dubbo-samples-nacos-configcenter、dubbo-samples-nacos-override、dubbo-samples-nacos-tagrouter，这些工程都有调用nacos的ConfigService.publishConfig(dataId, group, content)，跟踪发现这些工程没有调用NacosDynamicConfiguration、NacosMetadataReport的getChangeType方法，也没有调用AbstractConfiguratorListener的process方法。 目前我测试的案例只有内置zk的provider才会用到AbstractConfiguratorListener，我没看到dubbo provider内置其他注册中心的实现方法。 如果只是内置zk采用AbstractConfiguratorListener，那么创建节点的事件内容肯定是dubbo客户端ip，这个东西解析了也没有什么用处，没有weight等参数还是会被丢弃掉。 我感觉这个issue貌似没有什么用，如果上述判断没有问题，两个PR应该可以关掉了。 -- 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]
