hello-stephen commented on PR #67119: URL: https://github.com/apache/doris/pull/67119#issuecomment-5691846895
<!-- pr-67119-risk-acceptance-2026-09-16 --> 风险判断与合入决定： - Review 指出的场景真实存在：相同 PR head 可能对应不同的 base/测试合并提交；只核对 head SHA 不能完整证明当前验证目标。 - 近期数据表明该场景概率较低。2026-09-01 至 2026-09-15 的 Code Formatter 共 1,939 次 PR 运行，其中同一 head 的额外事件为 72 次（3.71%）；间隔超过 30 分钟、较可能涉及 base 或目标变化的只有 8 次（0.41%）。#67413 已证明不同测试合并目标可以出现；另有旧事件排队约 18.8 小时后与新事件同时开工，说明严重排队时所需的时序条件也真实存在，但样本中未观察到由此造成的错误合入。 - 命中后的风险主要是旧目标检查可能在同一 head 上产生误绿或误红，属于 CI 门禁正确性风险；本 PR 只修改四个 GitHub Actions workflow，不涉及 Doris 产品代码、数据或运行时行为。 - 这是现有 workflow 的残余风险，并非本 PR 新引入的回退。当前 master 在 base 变化后重跑旧 workflow 同样可能验证旧目标。本 PR 已解决不同 SHA 之间的反向误取消，并通过实际运行证明旧 head 任务会在 checkout 前停止。 - 接受该低概率 P2 作为已知限制：本 PR 提供的是“PR + 验证 SHA 隔离，以及开工时拒绝不同 head 的过期任务”，不宣称实现跨所有 base/head 组合的严格 latest-only。完整 validation-target fencing 后续单独处理。 合入前将把最新 master 同步到本分支，只接受新 head 上四项目标检查的终态通过；随后再处理 TeamCity 状态、Ready 状态和本次 review 的显式风险接受。 -- 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]
