AlbumenJ commented on issue #12337:
URL: https://github.com/apache/dubbo/issues/12337#issuecomment-1569457416
> 好的，了解了 我看了下最新版3.2.2 版本代码，在多机房注册中心时候，这边这个配置刷只会生效到一个机房吧？
第一个机房配置刷新后从新暴露设置到bounds中，后面机房循环进来更新时bounds中获取的exporterInvoker.getUrl()其实是之前更新过的，比较之后作为没有变更对应配置项就不会被更新。这样导致有时候调用的是配置生效的，有时候调用配置还是旧的
>
> ```
> public synchronized void doOverrideIfNecessary() {
> final Invoker<?> invoker;
> if (originInvoker instanceof InvokerDelegate) {
> invoker = ((InvokerDelegate<?>)
originInvoker).getInvoker();
> } else {
> invoker = originInvoker;
> }
> //The origin invoker
> URL originUrl = RegistryProtocol.this.getProviderUrl(invoker);
> String key = getCacheKey(originInvoker);
> ExporterChangeableWrapper<?> exporter = bounds.get(key);
> if (exporter == null) {
> logger.warn(INTERNAL_ERROR, "unknown error in registry
module", "", "error state, exporter should not be null", new
IllegalStateException("error state, exporter should not be null"));
> return;
> }
> //The current, may have been merged many times
> Invoker<?> exporterInvoker = exporter.getInvoker();
> URL currentUrl = exporterInvoker == null ? null :
exporterInvoker.getUrl();
> //Merged with this configuration
> URL newUrl = getConfiguredInvokerUrl(configurators, originUrl);
> newUrl =
getConfiguredInvokerUrl(getProviderConfigurationListener(originUrl).getConfigurators(),
newUrl);
> newUrl =
getConfiguredInvokerUrl(serviceConfigurationListeners.get(originUrl.getServiceKey())
> .getConfigurators(), newUrl);
> if (!newUrl.equals(currentUrl)) {
> if (newUrl.getParameter(Constants.NEED_REEXPORT, true)) {
> RegistryProtocol.this.reExport(originInvoker, newUrl);
> }
> logger.info("exported provider url changed, origin url: "
+ originUrl +
> ", old export url: " + currentUrl + ", new export url:
" + newUrl);
> }
> }
> ```
这个配置在多注册中心的时候确实是有问题的，可以帮提个 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]
