dubbo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From khanimteyaz (GitHub) <git...@apache.org>
Subject [GitHub] [incubator-dubbo] khanimteyaz commented on pull request #2920: enhance unit test
Date Fri, 07 Dec 2018 15:18:50 GMT
Do you feel it below mention way could be a better approach 
URL provider1=URL.valueOf("dubbo://")
               .addParameter("methods", "getXXX")
               .addParameter("group", "group1")
This might give better readability and might prevent accidental mistake of placing url escape

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

View raw message