terrymanu commented on PR #40064: URL: https://github.com/apache/shardingsphere/pull/40064#issuecomment-6009980266
### Result **Review Result: Mergeable** Yes—ready to commit the XML-driven SQL E2E coverage and documentation. No blocking code-review findings. The candidate is unchanged since verification. ### Evidence - Two JDBC DistSQL E2E cases passed with zero failures, errors, or skips, including cleanup. - All 22,544 existing test parameters remain unchanged. - Scoped Spotless, Checkstyle, and repository RAT checks passed. - Removed `ShardingDistSQLTest`; retained bilingual JDBC documentation and corrected the Proxy-only restriction. - Independent defensive-code review found no issues. ### Coverage Reviewed all 11 task-owned files against baseline `0a1ad58db7f4fa04712b28c9e1bbc65557375518`: scenario routing, XML/YAML fixtures, and bilingual documentation. Completed behavior, impact, and runtime review with no unresolved code findings. This is local code readiness; CI was not reviewed. GraalVM Native validation remains incomplete. -- 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]
