terrymanu commented on PR #39835: URL: https://github.com/apache/shardingsphere/pull/39835#issuecomment-5653804519
### Result **Review Result: Mergeable** Blocking Issues: 0 Required Changes: None Review Recommendations: No changes recommended. 修复范围明确,未改变公共 API、SPI 或既有 fallback 语义。 ### Evidence - Hive/Presto 列查询增加物理 `TABLE_SCHEMA` 过滤,避免跨 schema 混合。 - catalog/schema 每次加载只获取一次。 - 批量查询普通 VIEW,并正确设置 `TableType.VIEW`。 - Hive `string`、`varchar`、`char` 及 Presto `varchar`、`char` 正确标记为 case-sensitive。 - Hive fallback 行为保持不变。 - 新增断言覆盖 schema 参数、TABLE/VIEW 和字符串大小写属性。 ### Coverage - 类型:Standalone local candidate - 基线:`aba4bcfc2cc9b01a95cecbd21c8b9bee8286be7c` - 审查文件:4 个 - Root-cause、blast-radius、tests/runtime、convergence 审查全部完成。 - 没有未解决的证据缺口。 - 结论仅针对代码范围;未审查 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]
