dodjdnh commented on PR #3551: URL: https://github.com/apache/dubbo-go/pull/3551#issuecomment-5602386294
> 在检查指标实现细节时发现： `interface` 标签已经使用了 `url.Service()` ，因此我建议也在 `rpc.service` 中使用相同的标记，这样两个信号就能保持一致。目前的测试是针对组合后的组/接口/版本服务密钥设计的，这些测试也需要相应地进行修改。 > > 我很乐意根据管理员映射要求来审核此次更新。 明白了，这个方向我赞同。直接复用 url.Service() 确实更合适，这样 rpc.service 和 Metric 的 interface 使用同一套取值来源，也不需要再对 ServiceKey() 做额外解析。 我这边 Dubbo Admin P0 会按下面这个关系继续做 Trace -> Metric 映射： Trace rpc.service == Metric interface 如果 #3551 后续按 url.Service() 调整，相关 Provider / Consumer + group/version 的测试预期也一起更新即可，确保配置了 group/version 后 rpc.service 仍然只保留接口名。 感谢确认，后续如果有修改我也可以继续从 Admin 映射这边一起验证。 -- 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]
