luwei16 commented on PR #68792: URL: https://github.com/apache/doris/pull/68792#issuecomment-6075621247
PR: https://github.com/apache/doris/pull/68792 两轮审查已收敛：Blocker 0、Major 0、Minor 0、Nit 0。主审及三名交叉审查代理覆盖全部 6 个变更文件、三点 diff 和相关上下游调用链。审查过程未修改产品代码。 | 关键检查点 | 结论 | | --- | --- | | 目标与证据 | 非法 Boolean 和 snapshot/reset 参数均明确拒绝；单测覆盖错误消息、元数据及本地 offset。 | | 修改范围与复用 | 仅两处生产校验；复用严格 Boolean 解析器，不改变通用属性解析行为。 | | 并发与锁 | 沿用创建流程的读锁；扫描参数不可变，无新增锁或共享状态修改。 | | 生命周期 | 属性错误作用于未发布的新 Stream，不影响既有对象。 | | 配置 | 无新增配置；省略属性仍默认 false。 | | 兼容性 | 有意拒绝原先被忽略的非法输入；合法大小写和空参数调用保持支持，持久化及协议不变。 | | 平行路径 | 本地/Cloud 创建、MOW/DUP 扫描及 SELECT/INSERT 均经过相应公共校验。 | | 条件与错误处理 | 判断对应可达的非法用户输入；checked FE 与 unchecked Nereids 异常符合各自调用链。 | | 测试覆盖 | 直接校验、DDL、SQL 分析及负向回归均有覆盖；运行限制见下文。 | | 测试结果正确性 | 无新增或手写 .out；负向用例检查异常，正向 SQL 仅作 smoke 检查。 | | 可观测性 | 错误消息明确属性名或读取模式；无需新增指标。 | | 事务、持久化与恢复 | CREATE 校验在 ID 分配及 Cloud prepare/commit 前；扫描校验在 afterAnalyze offset hooks 前；日志和恢复格式不变。 | | 数据写入与原子性 | 非法 INSERT 在事务开始前失败；合法路径沿用既有写入与 offset 原子性逻辑。 | | FE/BE 变量传递 | 无新增跨端变量或协议字段。 | | 性能 | 仅语句级常量时间检查，无热循环或额外 RPC。 | | 其他问题 | 未确认其他缺陷，无未决疑点。 | 验证说明：本次为只读审查，未运行编译或测试。参考同一 head 的既有验证记录：39 个 FE 单测通过，Checkstyle 0 违规；新增 SQL 回归尚未在集群执行。 候选项处置：新增回归中的正向 SELECT 未比较结果行，失败 INSERT 未额外比较目标表及 Cloud offset。结合已有元数据/本地 offset 单测、既有有效结果回归及失败发生在事务前的调用顺序，将其视为可选覆盖增强和验证限制，不计为确认缺陷。 <!-- doris-repo-review:v1:begin --> ```yaml schema: doris-repo-review/v1 status: PASS pr: apache/doris#68792 commit: 2b4baf6fd41089b7151cacadb2831f879603c827 base: 038e2274ef678c44f85214e31da6d3e1d48a794f reviewed_at: 2026-10-09T04:53:55+00:00 reviewer: luwei16 model: gpt-6.1-sol effort: xhigh findings: {blocker: 0, major: 0, minor: 0, nit: 0} rounds: 2 converged: true ``` <!-- doris-repo-review:v1:end --> -- 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]
