dodjdnh commented on PR #3724: URL: https://github.com/apache/dubbo-go/pull/3724#issuecomment-5618493085
> `rpc.service` 改成 `url.Service()` 有利于和 Metrics 的 `interface` 对齐，但这会丢失 group/version 维度。当前 PR 又没有增加等价的 Trace attributes，导致同一 interface 的不同 group 无法区分。建议同时增加 `dubbo.group`、`dubbo.version`，并由 Admin 使用 interface + group + version 查询。 谢谢建议，我会在这个 PR 中继续保持 rpc.service 使用 url.Service()，并为 Server/Client Span 增加独立的 dubbo.group 和 dubbo.version 属性，避免 group/version 信息丢失，同时补充对应测试。 Dubbo Admin 当前第一阶段暂时不会将 group/version 作为 Trace → Metric 的查询条件。主要是为了兼容当前 Dubbo Java Trace：Java 的方法级 Metric 已有 group/version Label，但 Trace 目前并没有稳定输出对应属性。因此第一阶段 Admin 会先使用 interface + method + side + 时间范围 完成关联。 dubbo.group / dubbo.version 先在 dubbo-go Trace 中作为独立维度保留下来，后续再推进 Admin 和 Java 侧的精确 group/version 关联。 -- 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]
