terrymanu commented on PR #40124: URL: https://github.com/apache/shardingsphere/pull/40124#issuecomment-6094019887
### Result **Review Result: Mergeable** Upgraded ZooKeeper from `3.9.5` to `3.9.6`, changing only three version values across two files. ### Evidence - [pom.xml](/Users/zhangliang/IdeaProjects/shardingsphere/pom.xml:113) updates the managed version; [LICENSE](/Users/zhangliang/IdeaProjects/shardingsphere/distribution/proxy/src/main/release-docs/LICENSE:304) updates the ZooKeeper/Jute entries. - All 51 focused tests passed with the updated POM, including 13 tests using a real ZooKeeper server. Runtime verification confirmed both dependencies are `3.9.6`. - Only ZooKeeper/Jute changed among 114 resolved dependencies. POM Spotless and repository-wide Apache RAT checks passed with exit code 0. Independent review found no issues; no defensive code was added. ### Coverage Standalone local candidate: original task baseline `9d31ee552533`, current HEAD `22ff95e9292e`; scope includes only the local changes to the two files above. The implementation follows the user-approved dependency and license update scope, introducing no new project APIs or configuration commitments. Both areas completed root-cause, impact, test/runtime, and convergence reviews with no unresolved blockers. Checkstyle does not apply to the changed POM/LICENSE files; local Spotless covered the only applicable POM. This verdict covers code readiness only; 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]
