XiaoFeiASK commented on PR #1133: URL: https://github.com/apache/dubbo-go-samples/pull/1133#issuecomment-5301798924
新增修改: - graceful_shutdown/go-server/cmd/main.go 问题:为测试构造内部 ShutdownConfig,使用 NewOptions / SetShutdownConfig / server.SetServerShutdown,偏离 sample 应展示的公开 API 用法。 修复:恢复为直接 dubbo.WithShutdown(...) 传入 graceful shutdown options,并使用 ins.NewServer() 创建 server。 影响:sample 继续展示推荐的实例级公开配置方式,避免用户参考内部配置注入写法。 - integrate_test.sh 问题:自然 graceful shutdown 的 reject 窗口过短,独立 probe 可能在 listener 关闭后才发起,只得到 connection refused,无法证明请求被 provider graceful-shutdown filter 拒绝。 修复:将 graceful_shutdown 集成测试拆分为两段:第一段验证 in-flight 请求在 shutdown 后完成;第二段启动带 -reject-request=true 的 server,使用独立短连接 probe 稳定验证框架拒绝路径。 影响:同时满足 in-flight drain 与 framework reject path 的行为验证,避免 CI 依赖短暂时序窗口产生 flake。 -- 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]
