englefly commented on PR #67549:
URL: https://github.com/apache/doris/pull/67549#issuecomment-6165119780
按交接文档把**第 6 节（撤销 chosen 路径）**做完，并清掉第 2 节与第 3 节的遗留项。
**6.3 撤销清单（全部完成）**
- `build()`：删除 chosen 兜底整段，只描述**第一个 logical 表达式**；没有 logical 表达式时按组状态报
invalid（保留 `firstLogicalExpression == null` 的处理）。
- `representativeExpression()`：删除 `getChosenGroupExpression()` 分支；第一个
logical → 没有 logical 时才退到第一个 physical / enforcer。
- `Group.getChosenGroupExpression()`：删除（含那个永远匹配不到的 logical 循环）。
- `Group.setChosenGroupExpressionId()`：删除本会话加的 `this.hboStructInfo = null;`
/ `hboChainInfo = null;`。
- **保留**：纯物理 group 的处理、`PhysicalDistribute`/`PhysicalAssertNumRows`
透明、`LogicalFileScan`/`PhysicalFileScan` 分支。
**第 2 节**：`retryCtx`/`retried` 随分支删除而消失；`skipReasonOfPlanNode` 的 javadoc 改为
"Empty only when the description is valid, or when the group of the node is not
reachable at all."
**第 3 节遗留**：`copyStatsAndGroupIdFrom` 不再写入**空 group id**（两条通道都空的节点会传出一个不可解析的
id）；`AbstractPhysicalPlan` 该方法的 javadoc 也补上了。
**6.4 单测**：新增 `testTheStructInfoDoesNotDependOnTheChosenExpression`（同一计划建两个
memo，其中一个记录一个不属于任何表达式的 chosen id，断言 canonical / fingerprint 完全相同）→
`GroupStructInfoCompositionTest` **8/8 通过**。
（6.4 的第二条"纯物理 group 仍可描述"我尝试用测试手工搭一个纯物理 group，但 `ctx.visited` 是按结构判等的，手搭
group 会误判 `sharing=group 0 reached twice`，所以没有留下这条单测；该路径由实测覆盖：`count(*)` 走
`PhysicalStorageLayerAggregate` 时其 group 只有物理表达式，仍拿到 `type=exact`。）
**6.5 验收**
- 6 行矩阵（本机集群，BE 存活）：普通 join / `order by` / `limit` / **`order by … limit`**
全部 `type=exact` 且指纹相同（`9eeae21f…`）——第 4 行已从 `no group for PhysicalHashJoin`
修好；左孩子带 `limit 10` 的一行指纹不同（`fb866bf3…`，`L{10,0}` 进了身份），带 TopN 的一行与普通 join 相同 ✓。
- `query_p0/hbo` 正在重跑（上一次 19/19 失败是因为 BE 中途崩了、报 `No backend available as
scan node`，与本次改动无关）；结果我会在下一条评论补上。
--
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]
