FlyingZC commented on PR #40206:
URL: https://github.com/apache/shardingsphere/pull/40206#issuecomment-6309026559
### Result
Review Result: Mergeable
未发现阻塞问题或需要追加的修改。结论仅针对代码范围；未审阅 GitHub Actions。
### Evidence
- PostgreSQL 19 COPY、SELECT、WITH 官方手册及对应原始语法支持所审查的解析形式。
- 35 个新增 SQL 均有配套预期；11 个隐式 FETCH 预期准确断言默认值 1、模式标记及源文位置。
- Mac / JDK 21 完整 Unit/IT：30,233 tests，0 failures，0 errors，124 skipped；包括
PostgreSQL ParserIT 1,898 项、非法 SQL 35 项、行数 token 测试 8 项。
- 构建、测试、Spotless、Checkstyle、RAT 均退出 0；验证代码与公开提交内容一致。
### Coverage
- 完整公开 PR #40206：head
8dbe34adfd8ce34d63fd04e26ad6b3234665c83c，base/merge-base
a726e41a5744cdc7a0cc1bc0bea90651722b39e1。GitHub 与本地三点差异的 13 文件一
致，工作区干净。
- 覆盖全部 13 文件、29 改动块：COPY 语法边界、词法重复声明、FETCH AST/token、CTE JAXB/断言及全部
SQL/预期配对。完成三个发现视角和最终收敛，无未解决证据缺口。
- 依据当前 CODE_OF_CONDUCT.md、适用的 AI_POLICY.md、Parser/ParserIT 契约及 PostgreSQL
官方资料。现有执行层限制保持不变；openGauss 使用独立 parser/visitor。
- 两项样式检查覆盖全部适用 Java 文件：
-
features/sharding/core：RowCountToken.java、ShardingRowCountTokenGeneratorTest.java
- parser/sql/engine/dialect/postgresql：PostgreSQLStatementVisitor.java
-
test/it/parser：WithClauseAssert.java、ExpectedCommonTableExpressionClause.java
- 经受管适配器执行原生 Maven Wrapper，命令摘要：
./mvnw -B -ntp -pl
features/sharding/core,parser/sql/engine/dialect/postgresql,test/it/parser
-DspotlessFiles='<上述五文件精确路径正则>'
-Dcheckstyle.includes='<同五文件源根相对路径>' -Dcheckstyle.includeResources=false
-Dcheckstyle.includeTestResources=false spotless:check checkstyle:check -Pcheck
-T1
- Spotless 实际选择 1+2+2 文件，无缓存跳过；Checkstyle XML 确认同五文件全部选中，0 错误级违规。两项退出码均为 0。
- 其余 G4/XML 不在当前样式配置管理范围，两项检查均为 Not Applicable；无全局样式配置变更。
- 完整回归排除 test/e2e/sql；未追加数据库执行验证。原始验证结果已核对，未重复运行未失效的测试。
--
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]
