apache / incubator-hugegraph

A graph database that supports more than 100+ billion data, high performance and scalability (Include OLTP Engine & REST-API & Backends)
https://hugegraph.apache.org
Apache License 2.0
2.63k stars 517 forks source link

fix(pd-store): partially fix hstore backend core tests failure #2424

Closed VGalaxies closed 8 months ago

VGalaxies commented 8 months ago

Purpose of the PR

close https://github.com/apache/incubator-hugegraph/issues/2420

Main Changes

Some tests that are currently failing (not addressed in this PR):

  1. Tests related to OLAP in VertexCoreTest: Not adapted yet.
  2. Tests related to page in VertexCoreTest and EdgeCoreTest: Inconsistency in the definition of position between RocksDB scan iterator and Hstore.
  3. Some RamTableTest cases: The ramtable feature is not supported by the hstore backend.
  4. ...

Verifying these changes

Does this PR potentially affect the following parts?

Documentation Status

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (pd-store@499f8e7). Click here to learn what that means.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## pd-store #2424 +/- ## =========================================== Coverage ? 55.15% Complexity ? 1901 =========================================== Files ? 718 Lines ? 58294 Branches ? 7484 =========================================== Hits ? 32150 Misses ? 22974 Partials ? 3170 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

VGalaxies commented 8 months ago

@Pengzna, could you take a look at the errors related to MultiGraphsTest and EdgeCoreTest?

Pengzna commented 8 months ago

@Pengzna, could you take a look at the errors related to MultiGraphsTest and EdgeCoreTest?

Thanks your work, I'm working on it

VGalaxies commented 8 months ago

@Pengzna, could you take a look at the errors related to MultiGraphsTest and EdgeCoreTest?

Thanks your work, I'm working on it

enhance it in next PR~