Mulavar commented on pull request #1507: URL: https://github.com/apache/dubbo-go/pull/1507#issuecomment-939254671
有两个小疑问： 1. 看实现是把 cluster 结构体的语义部分都提取到了包名中，结构体命名只保留最基本的部分，比如 failbackClusterInvoker 就把 failback 提取到包路径上，结构体只保留后面的 clusterInvoker 部分，这个方式的可读性是不是更好呢？我举个例子，比如你用 goland 做跳转实现的时候，之前看到的是一堆 xxxClusterInvoke，现在看到的都是同名的结构体，需要进一步看路径名判断跳转，这样会不会反而增加了使用时的可读门槛？ 2. 常量命名要统一改驼峰形式么，可以定下来这个问题，后面统一调整一下？ -- 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]
