FlyingZC commented on PR #39914:
URL: https://github.com/apache/shardingsphere/pull/39914#issuecomment-5758446666
### Result
**Review Result: Mergeable**
最新 HEAD 已修复上一轮两个边界问题，未发现新的代码正确性阻塞项。
### Evidence
- `OracleDMLStatementVisitor.java:773-778` 在外层 `ORDER BY` 重建后保留 parameter
markers、variables 和 comments；对应 bind marker 测试已覆盖。
- `PLSQL.g4:581-594` 将 `%TYPE` 恢复为原有 `typeName` 路径，三段式名称仅用于普通集合类型。
- `OracleDDLStatementVisitor.java:2383-2389` 正确区分直接 `typeName` 与集合类型中的嵌套
`typeName`。
- Oracle Visitor `3/3`、Parser IT `1966/1966`、Rewrite IT `1019/1019` 均通过。
### Coverage
- Reviewed head：`1c7d3605e5f673b69850dc1f21d513122418ddf6`
- Base：`master` at `1bcf1101ad8bef3059674a463b740bda562887ee`
- GitHub 与本地 triple-dot 文件清单一致，共 9 个文件。
- 已完整审查 parenthesized WITH 与 qualified PL/SQL collection type 两个行为集群，以及最新
4 文件修复增量。
- 已覆盖 grammar、visitor、参数状态、元数据提取、兼容性、测试及 parser 内存风险。
- 本次为 Code Correctness Review；未查询或评价 CI。
--
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]
