luoning810 commented on issue #14185: URL: https://github.com/apache/dubbo/issues/14185#issuecomment-2112363034
Thank you very much for the response above. We are currently validating the upgrade from Dubbo 2.7 to Dubbo 3.2. DubboAdmin is indeed using a version adapted to 2.7, but I understand that the differences in interface-level dynamic configuration between the old and new versions mainly lie in whether the key includes the group and version. Even if we remove the group and version from the key, Dubbo still does not work properly. I tried to analyze the issue and believe the problem occurs in the following areas: <img width="1199" alt="image" src="https://github.com/apache/dubbo/assets/22349611/b528648a-fa4a-4442-b3cc-0beeddb22c89";> When referenceConfigurationListener.getConfigurators() is not empty, the overriddenURL may need to add group and version parameters so that AbstractConfigurator#isV27ConditionMatchOrUnset(overriddenURL) returns the correct result when comparing the configServiceKey. Currently, line 195 always returns false because the url is the overriddenURL mentioned above, which only includes the application name and does not contain the group and version. <img width="1410" alt="image" src="https://github.com/apache/dubbo/assets/22349611/5aebc388-6386-4b57-ace2-084e6a2507f0";> -- 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]
