[GitHub] [dubbo] chickenlj commented on issue #8325: Dubbo 2.7.12 consumer没有配置retries时，没有使用provider配置的retries

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

chickenlj commented on issue #8325:
URL: https://github.com/apache/dubbo/issues/8325#issuecomment-953576106


   > PS：
   > 
   > 1. 2.5.10、2.6.10无此问题
   > 2. 2.7.12下，dubbo.provider.cluster=failfast也是同样
   > 
   > 原因：2.7下org.apache.dubbo.registry.integration.RegistryDirectory#mergeUrl中，将提供者参数合并到overrideDirectoryUrl的代码被去掉了。
   > 
   > 撇开问题本身，这里其实有点尴尬的，因为retries、cluster更像是对一个服务所有提供者维度的，而不是针对一个服务具体某个提供者维度的参数。比如，一个10个提供者，它们注册的retries、cluster值不同。。
   
   我认为我们应该对能支持的场景做一些限制，不太可能无限的对所有可能的部署场景都提供支持。比如 `一个10个提供者，它们注册的retries、cluster值不同` 这种情形在 registryes应该就不能被支持，因为 retries 在框架中是工作在 ClusterInvoker - 也就是集群层面。


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