You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@dubbo.apache.org by "mzorro (GitHub)" <gi...@apache.org> on 2019/08/21 02:19:49 UTC

[GitHub] [dubbo] mzorro opened pull request #4899: [Dubbo-4882] Add support for overriding Map properties in AbstractConfig.refresh

Solution for issue #4882 
Add support for overriding Map properties in AbstractConfig.refresh, for example if we have configuration `dubbo.application.parameters=a=1&b=2`, then `ApplicationConfig.refresh` will do `parameters.put("a", "1");parameters.put("b", "2")`.

[ Full content available at: https://github.com/apache/dubbo/pull/4899 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org

[GitHub] [dubbo] chickenlj commented on issue #4899: [Dubbo-4882] Add support for overriding Map properties in AbstractConfig.refresh

Posted by "chickenlj (GitHub)" <gi...@apache.org>.

change format to `dubbo.application.parameters=[{a:1}, {b:2}]`

[ Full content available at: https://github.com/apache/dubbo/pull/4899 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org

[GitHub] [dubbo] cvictory closed pull request #4899: [Dubbo-4882] Add support for overriding Map properties in AbstractConfig.refresh

Posted by "cvictory (GitHub)" <gi...@apache.org>.

[ pull request closed by cvictory ]

[ Full content available at: https://github.com/apache/dubbo/pull/4899 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org

[GitHub] [dubbo] mzorro commented on issue #4899: [Dubbo-4882] Add support for overriding Map properties in AbstractConfig.refresh

Posted by "mzorro (GitHub)" <gi...@apache.org>.

> change format to `dubbo.application.parameters=[{a:1}, {b:2}]`

@chickenlj why do you think this format is more readable? It's not even standard JSON.

[ Full content available at: https://github.com/apache/dubbo/pull/4899 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org