chandrakiranbolla1 commented on issue #2954: URL: https://github.com/apache/incubator-hugegraph/issues/2954#issuecomment-3948940612
I’ve made significant progress on upgrading HugeGraph to TinkerPop 3.8.0. Completed so far: - Removed gremlin-shaded:3.5.1 - Migrated traversal optimize layer (Step generics, Aggregate changes) - Updated ConditionP to use PBiPredicate - Adapted HugePrimaryKeyStrategy for Mutating API changes - Migrated driver classes (RequestMessage, ResponseMessage, Tokens) to gremlin-util - hugegraph-core builds and tests pass successfully Verification: mvn -pl hugegraph-server/hugegraph-core -am test -Drat.skip=true → BUILD SUCCESS Full project build: mvn clean install -Drat.skip=true Build proceeds successfully through core and API modules. The remaining failure is in hugegraph-test, due to moved/removed TinkerPop test classes (e.g., StoreTest) in 3.8. Question: Should I continue aligning HugeGraph’s test suite with TinkerPop 3.8 changes, or is there a preferred intermediate target version (e.g., 3.6/3.7)? I’m happy to proceed and open a draft PR once test alignment is complete. @imbajin @zyxxoo — would appreciate your guidance on whether targeting 3.8 aligns with the current roadmap. -- 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]
