terrymanu commented on PR #40298: URL: https://github.com/apache/shardingsphere/pull/40298#issuecomment-6283785567
### Result Review Result: Mergeable No blocking issues or recommended additional changes remain. ### Evidence The current-source build using JDK 21 succeeded. All tests passed without failures, errors, or skips: 31 cases across six Agent groups, 368 SQL cases, four real MCP container cases, and 19 host-side metadata cases. Groovy inline expressions, Calcite/Janino, JDWP, premain on both architectures, and Proxy startup in default and Agent modes were verified. git diff --check passed. Spotless and Checkstyle do not apply to these Dockerfiles. No mandatory Apache RAT execution condition was triggered. ### Coverage Standalone local candidate against baseline 4fd67614f53956eef0a78eaa718bd00dcb3929f8. All five Dockerfiles were reviewed: official Agent/Proxy, the general Proxy test fixture, and Agent JDBC/Proxy test images. Changes only replace JDK images with same-major-version JRE images, preserving the JDK 21 build baseline and existing bytecode targets. Behavior, contract impact, testing/runtime, and convergence reviews were completed. Two consecutive rounds of independent two-reviewer assessments and coordinator verification reached consistent conclusions, with no unresolved differences or evidence gaps. This conclusion covers code correctness only. Remote CI was not reviewed. -- 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]
