#39534 [Fix] Aggregate all IPC weight update responses
原始 PR · 作者 bojiang-li · 合并时间 2026-09-15 14:25
缺陷修复
重要性 5.44
洞察度 5.00
IPC 权重更新改为归并全部响应，避免部分失败误报成功
值得精读，但不需要逐行推敲——这是一次 3 行改动、契约语义却相当关键的修复。建议重点看三点： 1）为什么 `(await communicator(obj))[0]` 是反模式：它把“多 rank 结果集”降级成单点判断，顺序敏感只是症状，根因是缺少统一聚合点； 2）`merge_results` 同时承担成功判定与诊断信息拼接，副作用（清缓存、推进版本号）被统一收敛到 `success` 之后，这个“先归因、再产生副作用”的顺序值得在其它控制面接口里复用； 3）维护者删掉 mock 单测的取舍：如果团队正在跑 RL 权重同步，建议自行补一次“部分 rank 失败”的真实场景验证（含 pause 分支），并确认上层客户端对失败的处理是重试还是终止。