Alanxtl commented on PR #3390: URL: https://github.com/apache/dubbo-go/pull/3390#issuecomment-4657457481
这个方向我觉得是对的，不过如果这次要把 `AddParam` 的语义问题作为 #3248 的一部分来改，建议把现有调用点也一起梳理完。 现在 PR 只是新增了 `AppendParam`，然后把 `AddParam` 标成 deprecated，但代码里还有不少内部调用仍然在用 `AddParam`。对于 `registry`、`registryType`、`tag`、`peer`、一些开关类参数这种单值语义更明显的地方，应该改成 `SetParam`；只有确实需要保留多个值的地方才改成 `AppendParam`。 否则这个 PR 更像是只补了一个新 API，实际的误用风险还在。建议这次把所有的调用点一起改完 -- 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]
