eldenmoon commented on PR #69995: URL: https://github.com/apache/doris/pull/69995#issuecomment-6599017027
## 对前述评审的归因及合入建议更正 本条更正[初次评审](https://github.com/apache/doris/pull/69995#issuecomment-6581204870)及[F-04 补充](https://github.com/apache/doris/pull/69995#issuecomment-6595258322)的归因和合入建议。依据已审查的 `ee6ce712a03`、`f95c5034e79`；后续合并 master 的 `e98b5630aa1` 不在本次复核范围内。 | 项目 | 与本 PR 的关系 | 对合入的影响 | |---|---|---| | F-01：escaped 类型 SQL 丢失显式 nested=false | 新功能的真实集成缺口；不是旧模式回归，但与本 PR 直接相关 | 已在 f95c5034e79 修复，不再阻塞 | | F-02：编码预校验重复加载/初始化 segment reader | 本 PR 新校验增加了可复用的工作；原有聚合也存在重复加载，不能把全部开销算成本 PR 新增 | 可选性能优化，未测量吞吐退化，不阻塞 | | F-03：两个独立路径的 CAST 仍按相同显示名分组 | 新路径身份与旧 CAST 收集逻辑的性能集成缺口；结果正确 | 可选性能优化，不阻塞 | | F-04：内部 sparse 名称误分类 | merge base 已有，未证明本 PR 引入或加重；作者已在 #70060 单独处理 | 不应计入本 PR 的新增缺陷或阻塞项 | 另外需要撤回 F-04 中过于确定的具体推导：此前写了“第一次 FULL 成功后字段不可见、第二次成功重写后不再保留物理值”。该场景未经运行验证；[作者的 ASAN 实测回复](https://github.com/apache/doris/pull/69995#issuecomment-6597234087)指出三 rowset 场景第一次 FULL 就在统计阶段失败。因此，那条“两次成功重写”的执行链没有得到证实，不能继续作为已确认的数据丢失复现。内部名称误分类本身与该具体执行链应分别判断。 **此前把 F-04 累加成“本 PR 存在 Blocker，REQUEST_CHANGES”的表述不恰当，现予以更正。** 问题本身的严重性与“是否由这个 PR 引入、是否应阻塞这个 PR”是两件事。就已审查的 f95c5034e79 而言，F-01 已修复，剩余 F-02/F-03 是非阻塞改进；没有确认本 PR 新增且尚未修复的 Major/Blocker，也没有确认由新 codec 引入的静默丢数据回归。本说明是归因更正，不是测试通过或对后续提交的复审声明。 -- 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]
