foghost

Results 24 comments of foghost

the CI error may caused by https://github.com/grpc/grpc-java/issues/10601

1. 兼容性问题:之前的实现 path 是 `/${appName}/${className}/${version}/${method}` ,新的实现会是 `/${appName}/${className}/${method}`,version 放到 header 里,会和之前的版本不兼容性 2. 目前的应用级实现是通过元数据接口补全了全部信息的,是不是可以和接口级实现进行统一,这样就不用前缀匹配了,避免出现 pixiu 这边 path 校验通过了,请求后端 dubbo 服务的时候发现没有方法这种 case 最终实现是不是可以统一成接口级的实现,应用级用元数据补全 url 就行了,这样实现起来更统一,对目前的代码改动量小

This should be a serialization issue,plz check your serializer doc for help,make a field transient in java server side should work

依赖确实有点容易循环,有些模块会把一模一样的配置再拷贝一个 struct 出来,只是为了避免循环依赖,例如 https://github.com/apache/dubbo-go/blob/bf5f2ddf40eaf9779355a692bab7842e290b54b2/config/metric_config.go#L45C55-L45C55, 是不是可以考虑做彻底点,把 config 只作为被依赖方,不再依赖其他模块

> 把配置都往下传 是怎么个传法,用 url ?

sounds good > > > 把配置都往下传 > > > > > > 是怎么个传法,用 url ? > > 1. 直接依赖,使用Init(opts ...option)方式把参数注入进去 > 比如ConsumerConfig依赖ReferenceConfig,之前是Init(rc *RootConfig),现在是Init(opts ...ReferenceOption)。这个需要我们对dubbo-go做好分层。 > 2. invoker链串联 > 使用URL传静态配置(config.Load时完成的所有配置),使用Invocation传动态配置(invoker链执行时查看)。invocation目前已经可以传复杂配置,URL也需要支持类似map[string]interface{}的复杂配置。每个invoker内部只管传进来的配置,不去config包拿。

I'll take [Metadata Center] and [Configuration Center], [Application Information], [Go Runtime] DingDing id ihja35w