eldenmoon commented on PR #65684: URL: https://github.com/apache/doris/pull/65684#issuecomment-6137021512
当前仍存在的三条问题，PR 评论中均已指出： 1. P1：ordered compaction 后 hidden 列可能返回错误值。 链接 singleton rowset 的 segment 不重写物理占位值；输出变成 range rowset 后直接读盘，TSO/VERSION 可能仍为 0，影响时间旅行、剪枝和 MIN/MAX。[读取分支 (line 1078)](/tmp/review-65684/source/be/src/storage/segment/segment.cpp:1078) 2. P2：整行 row-store 点查和 TopN 回捞返回 hidden 占位值。 JSONB 解码路径没有接入 rowset 的真实 VERSION/TSO；点查注释承认了限制，但仍直接返回错误值。[点查路径 (line 104)](/tmp/review-65684/source/be/src/service/point_query_executor.cpp:104) 3. P2：VARIANT benchmark 无法编译。 StorageReadOptions() 已删除，benchmark 仍使用默认构造。应改成 StorageReadOptions options(stats)。[调用位置 (line 1282)](/tmp/review-65684/source/be/benchmark/variant/benchmark_variant_segment.hpp:1282) VARIANT 重点检查结果： - 功能与语义：缺失 root 的值读取与物理元数据查询已分开；ADD、同名 DROP/ADD 的 UID 修正方向正确。NESTED SEARCH 使用当前 ReadSchema，避免绑定已删除的旧列。 - 兼容性：没有发现新增存储格式或 FE/BE 协议变化；旧数据缺失 sparse stream 时补空 MAP，doc compaction 缺失 bucket 时补 nullable VARIANT，调用链一致。 - 性能与可维护性：常量 reader 不进入共享缓存，避免请求间 VERSION/TSO 串值；接口职责更清楚。未做性能测量。 - 测试：有针对性的 BE 单测；新增 8 个回归 suite 的 81 个 qt 标签均有输出块。仍缺少 VARIANT schema evolution 后完整 compaction 的端到端覆盖。 -- 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]
