chaojixinren commented on PR #1536: URL: https://github.com/apache/dubbo-admin/pull/1536#issuecomment-5384285760
> @chaojixinren 上述是我的一些个人拙见，可以按照您的设计进行实际的一些调整和修改 : ) 感谢详细的 review！ **已修复**： - sonic 已升级到 v1.15.2，解决 Go 1.26 编译问题 - 新增 error/cancel/degrade 事件类型，错误事件在对应 .end 事件之前发射 - 已在 Registration.CaptureContent 注释中明确外部 hook 走急切快照，lazyContent 仅供内置 hook 使用。导出懒序列化 API 涉及设计权衡，建议后续单独 issue 跟踪 - panic drain 在原 commit 就修了，handlers.go 增加 discardAgentOutput + 回归测试 - hooks.yaml 和 README 都加了完整事件列表文档 **说明**： - 认同 panic drain 修复与持久化层的关联 - 当前 hooks.yaml 只配置内置 hook 开关，不支持配置化 hook 注册（Registration 在代码里构造），所以 schema 暂无 events 字段需要 enum。如果未来支持配置化注册，确实需要加 enum 提供补全 -- 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]
