chaojixinren commented on PR #3635:
URL: https://github.com/apache/dubbo-go/pull/3635#issuecomment-5384112765
Hi @NeverENG 和 @1kasa,
感谢详细的代码评审！我已经修复了所有反馈的问题：
1. 池键中缺少路径 - 在 nacosClientPoolKey 中添加了 url.Path，现在不同路径的 URL（如 /nacos vs
/custom）会创建不同的池键。已添加测试覆盖。
2. 锁优化 - 将 credentialIDsMu 从 sync.Mutex 改为 sync.RWMutex，使用读写锁模式以提高并发读取性能。
3. ZooKeeper 超时不一致 - 修复了 metadata-report 的超时默认值，从硬编码的 15s 改为使用
constant.DefaultRegTimeout (5s)，现在与 registry 和 config-center 保持一致。
4. Location 处理 - 添加了文档注释说明多个逗号分隔地址的行为。
所有修改都在最新的 commit 中。有时间的话请再帮忙看一下，谢谢！
--
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]
