DMwangnima commented on PR #2504:
URL: https://github.com/apache/dubbo-go/pull/2504#issuecomment-1810487603

   > How about we make `MergeURL` a method of *URL?
   > 
   > Because currently, the behaviour or priority of attribute keys is not 
determined since it depends on the sequence of parameters - the first URL 
always gets high priority, but not all maintainers know about this rule, so if 
they don't correctly put these two URLs, the result is probably not what they 
expect.
   
   +1. We need to distinguish between baseURL and addURL.


-- 
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]

Reply via email to