Skip to content

Comments

feat: add hugegraph-ldbc#524

Draft
Zony7 wants to merge 3 commits intoapache:masterfrom
Zony7:ldbc-branch
Draft

feat: add hugegraph-ldbc#524
Zony7 wants to merge 3 commits intoapache:masterfrom
Zony7:ldbc-branch

Conversation

@Zony7
Copy link
Contributor

@Zony7 Zony7 commented Oct 6, 2023

Purpose of the PR

Main Changes

The aim of this PR is to enable the execution of LDBC SNB (Social Network Benchmark) tests on HugeGraph. To achieve this, I have made modifications to certain portions of the code related to the workload and Gremlin query statements. Additionally, I have authored a README document to provide clear instructions on the testing process. These changes ensure that the LDBC tests can be executed successfully.

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

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

@github-actions github-actions bot added the loader hugegraph-loader label Oct 6, 2023
@codecov
Copy link

codecov bot commented Oct 6, 2023

Codecov Report

Merging #524 (dd909d0) into master (0aadaa3) will decrease coverage by 0.06%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #524      +/-   ##
============================================
- Coverage     62.47%   62.42%   -0.06%     
+ Complexity     1868      895     -973     
============================================
  Files           260       91     -169     
  Lines          9436     4404    -5032     
  Branches        876      519     -357     
============================================
- Hits           5895     2749    -3146     
+ Misses         3157     1450    -1707     
+ Partials        384      205     -179     

see 169 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions
Copy link

github-actions bot commented Nov 5, 2023

Due to the lack of activity, the current pr is marked as stale and will be closed after 180 days, any update will remove the stale label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature loader hugegraph-loader

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants