代码评审清单
变更:PR-0184 用品申领预占
作者:刘洋
评审人:张伟、王芳
日期:0000-00-00
结论:修改两处后可以通过
一、评审结论
PR-0184 把行政协同平台的用品申领从「提交即扣减」改为「先预占 15 分钟,确认领取后再扣实数」。10 分钟内的重复点击沿用同一幂等键。张伟和王芳于 9月27日完成评审。6 项检查里 4 项通过,2 项需要修改。修改完成前不合入主干。刘洋计划 9月28日中午前提交修改,王芳当日复审。
二、变更概要
改动集中在申领服务和库存扣减。提交时写入状态「预占中」,周敏在柜子前确认后变为「已领取」,超过 15 分钟未确认则由每分钟任务改为「已释放」。预占期间可申领数量减少,释放后恢复。总部 420 人使用这条路径,预发已有 18 条用例,其中重复点击 4 条只产生一笔预占。
三、检查结果
四、必须修改
- 最后 1 件用品被两人同时预占时,库存扣减要以数据库里的条件更新为准,失败的一方返回「已被预占」。刘洋补一条并发用例。
- 自动释放每处理一笔,日志写下申领单号、用品编号和释放数量,便于周敏对账。
- 两处修改后由王芳复跑并发用例和释放用例,张伟看日志样例后再合入。
五、复审安排
复审窗口是 9月28日 15:00。若并发用例仍出现两笔预占成功,本变更移出 10月8日的发布,申领保持提交即扣减。张伟在评审记录上确认了这个退路。通过后的合入由周敏在主干保护规则下点合并,刘洋不能自行合并自己的变更。评审用时 40 分钟,对照的预发版本是 v1.3.0-rc.1,签字笔库存当时为 19 支。王芳留下了失败用例的两笔单号 SL-0000-031 和 SL-0000-032,供复审时对照库存是否回到 1。复审通过后,陈静把本清单归档到当月发布材料。