contrueCT opened a new pull request, #3263: URL: https://github.com/apache/hugegraph/pull/3263
## Purpose of the PR Part 3 of a three-PR upgrade series targeting `apache/hugegraph:master`. Upgrade TinkerPop to 3.8.1 and Groovy to 4.0.25 with compatibility gates. Merge order: [apache#3261](https://github.com/apache/hugegraph/pull/3261) (Java 17 foundation) → [apache#3262](https://github.com/apache/hugegraph/pull/3262) (query semantics) → the corresponding ASF PR for [community #233](https://github.com/hugegraph/hugegraph/pull/233) (TinkerPop upgrade). All three PRs target `master` directly. Merge this PR after [apache#3261](https://github.com/apache/hugegraph/pull/3261) and [apache#3262](https://github.com/apache/hugegraph/pull/3262). Until the prerequisites are merged, the Files changed tab includes their changes; the part-3-only diff is available in [community #233](https://github.com/hugegraph/hugegraph/pull/233/files). This series develops the Java/TinkerPop modernization work tracked in [apache#3117](https://github.com/apache/hugegraph/pull/3117) and [apache#3069](https://github.com/apache/hugegraph/issues/3069), with the target version advanced to TinkerPop 3.8.1. Source: [hugegraph/hugegraph#233](https://github.com/hugegraph/hugegraph/pull/233), submitted directly from `hugegraph:task/tp381-3-upgrade-validation`. The local validation statements below are carried over from that source PR; CI on this ASF PR head remains the merge gate. ## Main Changes - Upgrade Server and distributed modules to TinkerPop 3.8.1 and Groovy 4.0.25. Adapt predicate and step APIs, GraphSON/GraphBinary registration and Cypher result normalization. - Keep Gremlin Groovy as the default engine; optional GremlinLang transport migration is outside this PR. - Run Structure/Process compatibility steps for the new upgrade branch on Memory, RocksDB and HStore. Include provider lifecycle and feature scenarios instead of relying on skipped CI steps. - Update release dependency/license metadata in a separate commit so it can be reviewed separately from the implementation. - Align Netty modules through the 4.1.137.Final BOM, replacing TinkerPop 3.8.1's vulnerable 4.1.125.Final dependency and updating release metadata. ## Verifying these changes - [x] Already covered by existing compatibility, traversal and serialization tests. - Local validation: Java 17 formatting and all-module clean compile passed. Provider lifecycle checks passed on Memory and RocksDB, including reopening after context shutdown. Report-gate checks passed for all TP entry points, including missing, malformed, empty, fully skipped and stale reports. HStore lifecycle and the full suites remain CI merge gates. - Full API, Structure/Process and distributed HStore CI must pass on this PR head before merging. A green workflow with skipped TinkerPop steps is insufficient. ## Does this PR potentially affect the following parts? - [x] Dependencies - [x] Modify configurations - [x] The public API - [x] Other affects: client serialization and script compatibility - [ ] 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: `docs/upgrade-tinkerpop-3.8.md` and `docs/query-semantics.md`. -- 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]
