#53453 [KVConnector][P2P] Configurable unbound-store timeout and one-RTT rejection of a late fetch
原始 PR · 作者 liranschour · 合并时间 2026-09-15 10:38
P2P 卸载层超时可配,晚到 fetch 一 RTT 内被拒
值得精读,尤其是负责 KV offload / tiering / PD 分离部署的工程师。三个值得关注的设计点:一是用带剪枝的墓碑表 `_reaped_stores` 承接跨实例失败通知,复用既有 `finish_request` → `_finalize_outbound` 终态路径而不改协议、不加 session API,是把「超时」变成「可立即观测的失败」的干净做法;二是配置入口「先 `float()` 强转、再以 `NaN` 兜底统一报错」的模式,对任何从 `kv_connector_extra_config` JSON 原样 splat 进来的 tier 参数都适用,可直接照搬到其它 tier;三是 `_REAPED_ID_RETENTION_S` 与消费者 `_LOAD_TIMEOUT_S` 的隐式耦合是唯一需要持续盯住的设计点,建议后续要么加断言/测试,要么把二者关联起来。测试方面,`recording_drain` 断言「线上从未出现 abort_fetch」这种验证路径而非仅验证结果的手法,也值得其它失败路径测试借鉴。
参与讨论