luwei16 commented on PR #69494: URL: https://github.com/apache/doris/pull/69494#issuecomment-6459629929
[PR #69494](https://github.com/apache/doris/pull/69494) 审查通过。两轮主审与三个子代理交叉审查已收敛，Blocker / Major / Minor / Nit 均为 0。 | 关键检查点 | 结论 | |---|---| | 目标与正确性 | 裁剪按 wrapper 的扫描索引识别分区，修复有效 changelog tablet 被裁为空计划的问题。 | | 范围与复用 | 一行生产改动，复用现有 `getPartitionIndex()` 接口。 | | 并发与锁 | 沿用规划读锁；schema change 的分区索引和表索引切换受同一写锁保护。 | | 生命周期 | 固定 TSO 初始化无外部依赖；配置恢复和静态 mock 关闭覆盖异常路径。 | | 兼容性 | 普通查询、独立 changelog index 和复用 base tablet 均保持正确语义。 | | 平行路径 | 与 tablet 裁剪和物理扫描的索引解析一致；snapshot/stream 路径无遗漏。 | | 条件与错误边界 | 分区、谓词与 tablet 取交集正确；无效 ID 和冲突条件沿用空结果行为。 | | 测试覆盖与结果 | 覆盖 RANGE/LIST、两种索引形式、无 WHERE、多 tablet、负例、APPEND_ONLY 及 EXPLAIN 扫描 ID；未手写结果文件。 | | 可观测性 | 现有 EXPLAIN 足以定位扫描索引与 tablet，无需新增日志或指标。 | | 性能 | 遍历复杂度不变，仅增加每分区常数时间索引查找，无新增 RPC/I/O。 | 无新增配置项、事务/数据写入、持久化、存储格式、gensrc 或 FE–BE 协议变量变更。未发现其他问题或未决疑点。本次审查未运行构建或测试；已检查审查时当前 head 的 FE UT、P0、Cloud 和 NonConcurrent CI 成功状态。 <!-- doris-repo-review:v1:begin --> ```yaml schema: doris-repo-review/v1 status: PASS pr: apache/doris#69494 commit: 8598111a473274dc6288a7466fd5d65b3f3de414 base: 4751c5f593cf7f136f2b193621bd5eabde7dc404 reviewed_at: 2026-10-27T08:07:27+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]
