Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Load vertices before edges when check-vertex is true #206

Merged
merged 4 commits into from
Apr 28, 2021

Conversation

Linary
Copy link
Contributor

@Linary Linary commented Apr 23, 2021

No description provided.

@Linary Linary force-pushed the check-vertex-load-vertex-before-edge branch from 8c168c7 to 01ecbbb Compare April 23, 2021 04:22
Change-Id: Ibc262d20885509df8ad5a8e7364cd7a7b0cdbf33
Change-Id: I1c342424e81722755574e0430464ed50e0a99b81
@Linary Linary force-pushed the check-vertex-load-vertex-before-edge branch from 01ecbbb to 4d8903f Compare April 26, 2021 08:54
Change-Id: I7307611442f617ca5cbd1e8f7c0f2d1e3943891a
@Linary Linary force-pushed the check-vertex-load-vertex-before-edge branch from 4d8903f to d95d536 Compare April 27, 2021 04:16
@codecov
Copy link

codecov bot commented Apr 27, 2021

Codecov Report

Merging #206 (4575b93) into master (e6915d8) will increase coverage by 0.60%.
The diff coverage is 95.13%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #206      +/-   ##
============================================
+ Coverage     73.44%   74.04%   +0.60%     
- Complexity      848      871      +23     
============================================
  Files            80       80              
  Lines          3577     3641      +64     
  Branches        429      436       +7     
============================================
+ Hits           2627     2696      +69     
+ Misses          749      745       -4     
+ Partials        201      200       -1     
Impacted Files Coverage Δ Complexity Δ
...m/baidu/hugegraph/loader/executor/LoadOptions.java 74.41% <ø> (ø) 9.00 <0.00> (ø)
.../baidu/hugegraph/loader/task/ParseTaskBuilder.java 90.32% <ø> (ø) 14.00 <0.00> (ø)
...a/com/baidu/hugegraph/loader/util/MappingUtil.java 82.55% <0.00%> (-0.98%) 13.00 <1.00> (ø)
.../baidu/hugegraph/loader/util/HugeClientHolder.java 25.49% <50.00%> (ø) 5.00 <0.00> (ø)
...om/baidu/hugegraph/loader/metrics/LoadMetrics.java 86.58% <88.37%> (+8.01%) 28.00 <16.00> (+7.00)
...va/com/baidu/hugegraph/loader/HugeGraphLoader.java 92.38% <100.00%> (+1.20%) 45.00 <16.00> (+7.00)
...om/baidu/hugegraph/loader/mapping/InputStruct.java 93.02% <100.00%> (+11.54%) 22.00 <6.00> (+7.00)
...com/baidu/hugegraph/loader/metrics/LoadReport.java 96.77% <100.00%> (ø) 15.00 <0.00> (ø)
...om/baidu/hugegraph/loader/metrics/LoadSummary.java 91.80% <100.00%> (ø) 27.00 <2.00> (ø)
...baidu/hugegraph/loader/source/file/FileSource.java 70.76% <100.00%> (ø) 26.00 <1.00> (ø)
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e6915d8...4575b93. Read the comment docs.

Change-Id: If1f6102afe6b1ed090520aecd52a9641606f446d
@Linary Linary force-pushed the check-vertex-load-vertex-before-edge branch from b6688f6 to 4575b93 Compare April 27, 2021 06:12
@Linary Linary merged commit 05967f0 into master Apr 28, 2021
@Linary Linary deleted the check-vertex-load-vertex-before-edge branch April 28, 2021 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants