Aias00 commented on code in PR #982: URL: https://github.com/apache/dubbo-go-pixiu/pull/982#discussion_r3595630237
########## pkg/adapter/dubboregistry/registry/nacos/registry.go: ########## @@ -18,10 +18,10 @@ package nacos import ( - "github.com/nacos-group/nacos-sdk-go/clients" - "github.com/nacos-group/nacos-sdk-go/clients/naming_client" - nacosConstant "github.com/nacos-group/nacos-sdk-go/common/constant" - "github.com/nacos-group/nacos-sdk-go/vo" + "github.com/nacos-group/nacos-sdk-go/v2/clients" Review Comment: 已处理（commit `0e94c1ee`）。未保留 v1 兼容回退路径（会重新引入 v1 依赖，与本次迁移目的相悖），改为按你给的第二个方案： - 在 `docs/ai/registry.md` / `registry_CN.md` 顶部新增 **Nacos 2.x 服务端要求** 说明：v2 客户端不兼容 1.x 服务端，最低要求 2.x（建议 2.2.0+）。 - 明确 **gRPC 端口 = 主端口 + 1000**（8848→9848），并要求防火墙/容器映射/反代场景同时开放 `8848/TCP` 与 `9848/TCP`，否则运行期失联。`docs/ai/mcp/mcp.md` / `mcp_CN.md` 同步补充。 - 修正示例中过时的 v1 导入 `nacos-sdk-go/vo` → `nacos-sdk-go/v2/vo`。 真实 2.x 连接验证受限于 CI 无 Nacos 实例，未新增 e2e；关闭路径已用 mock 断言。 -- 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]
