search

bilibili / discovery

A registry for resilient mid-tier load balancing and failover.
MIT License
1.78k stars 400 forks source link

brpc c++ SDK 接口解析与 API 文档不一致 #70

Open wasphin opened 3 years ago

wasphin commented 3 years ago

brpc 客户端部分使用 fetchs 批量获取实例，但解析响应部分与文档说明以及实际运行结果不一致，麻烦确认一下看是那边的问题。

API 文档 https://github.com/bilibili/discovery/blob/master/doc/api.md#%E6%89%B9%E9%87%8F%E8%8E%B7%E5%8F%96%E5%AE%9E%E4%BE%8Bfetchs

以及 brpc 源码

https://github.com/apache/incubator-brpc/blob/2aea69107891060eeb98ec7ce61f1a1a8f033ada/src/brpc/policy/discovery_naming_service.cpp#L390

https://github.com/apache/incubator-brpc/blob/2aea69107891060eeb98ec7ce61f1a1a8f033ada/test/brpc_naming_service_unittest.cpp#L467

lintanghui commented 3 years ago

这个我们兼容下，discovery的开源版本是只暴露了一个instances字段 。 内网版本为了兼容还保留了 zone_instances 字段用于做机房级别的区分