contrueCT opened a new pull request, #3261: URL: https://github.com/apache/hugegraph/pull/3261
## Purpose of the PR Part 1 of a three-PR upgrade series targeting `apache/hugegraph:master`. Establish a Java 17 build and runtime baseline while retaining TinkerPop 3.5.1. Merge order: the corresponding ASF PR for [community #231](https://github.com/hugegraph/hugegraph/pull/231) (Java 17 foundation) → the corresponding ASF PR for [community #232](https://github.com/hugegraph/hugegraph/pull/232) (query semantics) → the corresponding ASF PR for [community #233](https://github.com/hugegraph/hugegraph/pull/233) (TinkerPop upgrade). All three PRs target `master` directly. This PR is the prerequisite for the query-semantics and TinkerPop-upgrade PRs. Source: [hugegraph/hugegraph#231](https://github.com/hugegraph/hugegraph/pull/231), submitted directly from `hugegraph:task/tp381-1-java17-foundation`. The local validation statements below are carried over from that source PR; CI on this ASF PR head remains the merge gate. ## Main Changes - Align compiler, test plugins, JVM options, launchers, Docker images and cluster-test process handling on Java 17. Use Groovy 2.5.23 as a temporary compatibility pin: 2.5.14 cannot read Java 17 classes. Part 3 replaces it with Groovy 4. - Carry the GraphId allocation/repair work already reviewed in [#163](https://github.com/hugegraph/hugegraph/pull/163), together with the Java 17 work from [#183](https://github.com/hugegraph/hugegraph/pull/183) and [#194](https://github.com/hugegraph/hugegraph/pull/194). - Declare Commons Configuration 2.10.1 and Commons Text 1.11.0 in the release LICENSE, matching the dependency inventory. - The current community master already contains upstream [apache#3157](https://github.com/apache/hugegraph/pull/3157) and [apache#3220](https://github.com/apache/hugegraph/pull/3220); these are baseline changes and are excluded from this PR diff. ## Verifying these changes - [x] Already covered by existing tests: Java 17 launcher contracts, process-wait helpers, backend/serializer registration, auth reflection, and ordered scans under the Gremlin sandbox. - Local validation: Java 17 formatting and all-module clean compile passed. A local HTTP fixture verified that the actual PD startup script waits through 503/401 responses, accepts readiness HTTP 200, and fails on persistent unauthorized responses. - CI results remain the merge gate. The Groovy compatibility pin is the new integration adjustment requiring attention beyond the previously reviewed inputs. ## Does this PR potentially affect the following parts? - [x] Dependencies - [x] Modify configurations - [ ] The public API - [x] Other affects: minimum build/runtime JDK and cluster test lifecycle - [ ] Nope ## Documentation Status - [x] `Doc - TODO`: coordinate the merge of [apache/hugegraph-doc#508](https://github.com/apache/hugegraph-doc/pull/508) with this upgrade series. - [ ] `Doc - Done` - [ ] `Doc - No Need` Repository documentation is included in `docs/BUILDING.md`, the README and PD/Store deployment guides. License declarations and the dependency inventory are updated for the Groovy compatibility pin. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
