Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Support uuid idstrategy #59

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 16, 2019
Merged

Support uuid idstrategy #59

merged 1 commit into from
Aug 16, 2019

Conversation

Linary
Copy link
Contributor

@Linary Linary commented Aug 12, 2019 •
edited by javeme
Loading

implement https://github.com/hugegraph/hugegraph/issues/617
Change-Id: Ie86f8a7753bf9e0cde92abf7ac3d7cf72d95769b

@codecov
Copy link

codecov bot commented Aug 15, 2019 •
edited
Loading

Codecov Report

Merging #59 into master will increase coverage by 0.17%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #59      +/-   ##
============================================
+ Coverage     75.18%   75.35%   +0.17%     
  Complexity      508      508              
============================================
  Files            94       94              
  Lines          2285     2301      +16     
  Branches        143      143              
============================================
+ Hits           1718     1734      +16     
  Misses          464      464              
  Partials        103      103
Impacted Files Coverage Δ Complexity Δ
...om/baidu/hugegraph/structure/schema/EdgeLabel.java 92.85% <100%> (+0.4%) 6 <0> (ø) ⬇️
.../baidu/hugegraph/structure/schema/VertexLabel.java 92.85% <100%> (+0.85%) 4 <0> (ø) ⬇️
.../baidu/hugegraph/structure/schema/PropertyKey.java 70.37% <100%> (+2.37%) 3 <0> (ø) ⬇️
...baidu/hugegraph/structure/constant/IdStrategy.java 57.14% <100%> (+2.14%) 2 <0> (ø) ⬇️
...m/baidu/hugegraph/structure/schema/IndexLabel.java 87.23% <100%> (+0.56%) 6 <0> (ø) ⬇️

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 d7c47b5...966bef2. Read the comment docs.

@Linary
Support uuid idstrategy
966bef2 
Change-Id: Ie86f8a7753bf9e0cde92abf7ac3d7cf72d95769b
@zhoney zhoney merged commit ea85a8b into master Aug 16, 2019
@zhoney zhoney deleted the uuid-idstrategy branch August 16, 2019 07:56
zhoney pushed a commit to zhoney/hugegraph-client that referenced this pull request Sep 19, 2019
@Linary @zhoney
Support uuid idstrategy (hugegraph#59)
e765996 
Change-Id: Ie86f8a7753bf9e0cde92abf7ac3d7cf72d95769b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@javeme javeme javeme approved these changes

+1 more reviewer

@zhoney zhoney zhoney approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Linary @javeme @zhoney