CAICAIIs commented on PR #3299: URL: https://github.com/apache/dubbo-go/pull/3299#issuecomment-4281115270
> YAML已经暴露了？我看先应该是只支持global不支持config的吧，通过yaml配置不是走的config吗 确实说法不准确，按 develop 现在的实现看，这 4 个新字段目前只在 global/runtime 侧生效，而本地 YAML 这条链路还会经过 global -> config -> global 的 compat，我看了下 config 当前确实还移除不彻底， config.Http3Config 还没有这 4 个字段，所以 YAML 下现在不会生效，如果后续把 Load/init 链路里对 config 的依赖彻底移除，这个问题会自然消失。因为本地 YAML 现在本来就是先反序列化到 InstanceOptions(global)，只是后面还会经过 global -> config -> global 的 compat 往返，才把 config 不认识的字段丢掉了 -- 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]
