chaojixinren commented on PR #1536: URL: https://github.com/apache/dubbo-admin/pull/1536#issuecomment-5653373987
@ambiguous-pointer 感谢 review！已更新这个 PR： - 移除了 develop 带入的无关内容，差异从 170 个文件缩减到 42 个，并同步了最新 ai 分支。 - `active` 中的条目已有 `finishInteraction()` 清理；补充了重复并发交互和错误退出测试，验证结束后条目释放、context 取消。 - 保留现有锁保护和注册快照机制，补充并发 Register/Emit/NeedsContent 及回调内注册测试，相关 race 测试连续运行 10 次通过。 - 补充 10 MiB 多字节 Payload 的截断测试，验证大小限制、JSON/UTF-8 有效性及未开启采集时的惰性行为。 - 增加启用 Hook 时的分配基准，暂未引入 sync.Pool 或无锁结构，后续可根据实际性能需求优化。 AI 全量短测试、静态检查和编译均通过，SonarCloud 检查也已通过。目前 PR 无合并冲突，麻烦再帮忙看一下，谢谢！ -- 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]
