Skip to content

fix: support new HugeGraph edge id format#349

Open
hutiefang76 wants to merge 1 commit into
apache:masterfrom
hutiefang76:codex/fix-hugegraph-edge-id-format
Open

fix: support new HugeGraph edge id format#349
hutiefang76 wants to merge 1 commit into
apache:masterfrom
hutiefang76:codex/fix-hugegraph-edge-id-format

Conversation

@hutiefang76

Copy link
Copy Markdown

Purpose of the PR

HugeGraph server now formats edge ids with 5 or 6 parts after the parent/child edge label change. The computer loader still called Edge.name() from the older client, which expects exactly 4 parts and can fail while loading edges for algorithms such as LPA.

Main Changes

  • Add HugeConverter.convertEdgeName() to read sort values from 4-part, 5-part, and 6-part edge ids.
  • Use the converter in LoadService instead of calling edge.name() directly.
  • Add regression coverage for both 5-part and 6-part edge id formats.

Verifying these changes

  • Trivial rework / code cleanup without any test coverage. (No Need)
  • Already covered by existing tests, such as (please modify tests here).
  • Need tests and can be verified as follows.
mvn -pl computer-test -am -Djacoco.skip=true \
  -Dtest=HugeConverterTest#testConvertEdgeNameWithFivePartEdgeId+testConvertEdgeNameWithSixPartEdgeId \
  -Dsurefire.failIfNoSpecifiedTests=false test

mvn -pl computer-test -am -Djacoco.skip=true \
  -Dtest=HugeConverterTest \
  -Dsurefire.failIfNoSpecifiedTests=false test

Does this PR potentially affect the following parts?

  • Nope
  • Dependencies (add/update license info)
  • Modify configurations
  • The public API
  • Other affects (typed here)

Documentation Status

  • Doc - TODO
  • Doc - Done
  • Doc - No Need

@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jun 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] The edge id is formatted by 5 instead of 4 parts

1 participant