dubbo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From tswstarplanet (GitHub) <git...@apache.org>
Subject [GitHub] [incubator-dubbo] tswstarplanet commented on issue #2959: fix a bug of service config
Date Fri, 14 Dec 2018 16:52:17 GMT
这个提交出于以下目的,ServiceConfig会先加载ProtocolConfig的配置,之后加载自身的配置。如果两者有相同的配置项的话,ServiceConfig的配置会覆盖ProtocolConfig的配置。但之前的这个条件判断,是在加载了ProtocolConfig以及ServiceConfig之后,又修改了ProtocolConfig的值。这应该是不合理的,因为单独改变ProtocolConfig的配置没什么意义。所以应该放到前面。
而map中的notify属性,放到前面或者后面应该都没关系。所以就一起放到了前面。

[ Full content available at: https://github.com/apache/incubator-dubbo/pull/2959 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org
Mime
View raw message