Files
a-share-analysis/docs/features/GOV-008-rocketmq-adoption/review/code-review.md

1012 B

代码 Review

第一轮

结论:changes_required。发现 2 个 Important:配置混入旧 Remoting Producer Group 和无效的客户端最大重试字段;RabbitMQ 残留断言只覆盖生产 Worker。

处置

  • 删除 ROCKETMQ_PRODUCER_GROUP 和 ROCKETMQ_MAX_RECONSUME_TIMES。
  • 明确最大投递次数属于服务端 Consumer Group 运维策略。
  • 扩展测试覆盖 Scheduler/Worker 的开发与生产四份模板,同时核对开发和生产连接地址边界。

最终复审

  • 四份 Scheduler/Worker 模板均移除旧 RabbitMQ 字段、旧 Remoting Producer Group 和无效的客户端最大重试字段。
  • 开发模板统一使用 127.0.0.1:9876 和 127.0.0.1:8081 且保持关闭;生产连接地址保持空值。
  • 最大投递次数归属服务端 Consumer Group 运维策略,文档和配置语义一致。
  • 环境 7/7、治理 5/5、MySQL 2/2、Schema 23/23 及 Schema 验证通过。

Critical、Important、Minor 均无未关闭项。

最终结论:passed