Mulavar commented on pull request #1507: URL: https://github.com/apache/dubbo-go/pull/1507#issuecomment-939256458
> > 这样会不会反而增加了使用时的可读门槛？ > > 我个人认为不会，因为对外访问一般是通过 extension.GetCluster 函数，该函数返回的是一个 cluster.Cluster 的 interface，用户需要通过 interface 再找到相应的 cluster 定义，一般不会直接使用 xxxCluster（Invoker 同理，一般都会先定位到 interface）。在现有目录下，每种 cluster 的结构和定义都更加清晰，且有更良好的扩充性，这是我想要做这次重构的初衷。 > > > 常量命名要统一改驼峰形式 > > 是的，Golang 中 `XX_YY_ZZ` 本身是不符合规范的，所以驼峰式命名我个人认为会好一些。但是本次 PR 只是重构 cluster 相关代码，不涉及到常量名重构，这部分内容可以以后在实现。 第一个我指的场景就是指 interface 跳转到实现的时候，之前跳转根据结构体命名可以直接找对应的，但现在结构体命名都一样了，他就需要根据你的路径去区分，我理解你做这个思路，拆分目录肯定是更有利的，我的疑问可能更集中在所有 cluster 和 invoker 的命名被简化了这一点上（语义被移到了包路径上，这是一个有点奇怪的事）。 第二个我建议要不在社区里公告一下，确定我们之后一定会统一成驼峰形式，这样也可以避免万一意见不统一又回过头来把这个pr的内容改回去了。 -- 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]
