Conversation
- Poll the Raft readiness endpoint before client tests - Accept only HTTP 200 as a ready PD response - Prevent authentication failures from passing startup checks
- Inherit the PD readiness wait from the foundation branch - Keep the query semantics changes unchanged - Preserve existing stack ancestry for downstream pull requests
- Exercise cleanup and close/reopen on the selected backend - Reject empty or entirely skipped reports through every TP entry point - Isolate RocksDB fixtures from source formatting and cover gate failures - Inherit PD readiness checks while preserving downstream stack ancestry
- Use the default single-disk layout for the lifecycle fixture - Keep data and WAL under the JUnit temporary directory - Prevent optional index mappings from escaping test-owned storage
- Document vertex and edge uniform-list feature flag changes - Preserve schema-defined LIST guidance and migration expectations - Align both architecture diagrams with TinkerPop 3.8.1
- Close committed fallback iterators on success and failure - Assert that count optimization retains the ordering barrier - Cover cleanup through the primary-key count fallback path
- Inherit committed count iterator cleanup from the query PR - Carry the strengthened ordering and cleanup regression tests - Preserve the upgrade stack and downstream commit ancestry
- limit backend count to traversal start scans - cover repeated scans, input bulk and explicit ids - document intermediate scan count semantics
- align predicate pushdown with supported conditions - preserve type operands through nested negation - test native type filters and document execution
- inherit intermediate scan count protection - include count boundary regression tests - keep the upgrade branch aligned with query semantics
This was referenced Oct 3, 2026
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #3263 +/- ##
============================================
+ Coverage 41.57% 41.67% +0.10%
- Complexity 7311 7400 +89
============================================
Files 793 795 +2
Lines 69106 69491 +385
Branches 9258 9339 +81
============================================
+ Hits 28730 28963 +233
- Misses 37098 37223 +125
- Partials 3278 3305 +27 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
25 tasks
- remove Commons from required checks - require the Java 17 memory Server check - retain analysis and license checks
- avoid claiming TinkerPop 3.8.1 in the foundation - describe shared TinkerPop and Kryo access needs - retain the test and runtime permission boundary
- describe V1 fallback with generic type metadata - keep all three client examples consistent - retain the supplied serializer and registry settings
- update both Store dependencies to Fastjson 1.2.84 - align the release license and dependency inventory - retain the Fastjson 1.x dependency contract
- inherit the Fastjson security patch - align required checks with Java 17 - retain consistent test JVM guidance
- inherit the Fastjson security patch - align required checks with Java 17 - retain consistent test JVM guidance # Conflicts: # install-dist/release-docs/LICENSE
- override the legacy client transitive dependency - retain the existing client and production versions - remove obsolete release inventory entries
- inherit the cluster Commons Text override - retain the query behavior and client version - align release dependency metadata
- inherit the cluster Commons Text override - retain the upgrade behavior and client version - align release dependency metadata # Conflicts: # install-dist/scripts/dependency/known-dependencies.txt
3 of 9 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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 (Java 17 foundation) → apache#3262 (query semantics) → apache#3263 (TinkerPop upgrade). All three PRs target
masterdirectly. Merge this PR after apache#3261 and apache#3262. Until the prerequisites are merged, the Files changed tab includes their changes; the part-3-only diff is available in community #233. This series develops the Java/TinkerPop modernization work tracked in apache#3117 and apache#3069, with the target version advanced to TinkerPop 3.8.1.Source: hugegraph/hugegraph#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
Verifying these changes
Does this PR potentially affect the following parts?
Documentation Status
Doc - TODO: coordinate the merge of apache/hugegraph-doc#508 with this upgrade series.Doc - DoneDoc - No NeedRepository documentation:
docs/upgrade-tinkerpop-3.8.mdanddocs/query-semantics.md.