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

fix: can't dynamic create graph if auth enabled #1708

Merged
merged 1 commit into from
Dec 27, 2021

Conversation

javeme
Copy link
Contributor

@javeme javeme commented Dec 24, 2021

Change-Id: I9f6c62d9eb152e7b6a45a078ae28f9a41828ae40

Change-Id: I9f6c62d9eb152e7b6a45a078ae28f9a41828ae40
@codecov
Copy link

codecov bot commented Dec 24, 2021

Codecov Report

Merging #1708 (b450ca6) into master (d213166) will increase coverage by 2.51%.
The diff coverage is 16.00%.

❗ Current head b450ca6 differs from pull request most recent head e8f5c1a. Consider uploading reports for the commit e8f5c1a to get more accurate results
Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1708      +/-   ##
============================================
+ Coverage     64.47%   66.99%   +2.51%     
- Complexity     6931     7103     +172     
============================================
  Files           423      423              
  Lines         35484    35488       +4     
  Branches       4948     4948              
============================================
+ Hits          22878    23774     +896     
+ Misses        10272     9318     -954     
- Partials       2334     2396      +62     
Impacted Files Coverage Δ
...in/java/com/baidu/hugegraph/core/GraphManager.java 38.33% <0.00%> (+1.17%) ⬆️
...src/main/java/com/baidu/hugegraph/HugeFactory.java 71.69% <0.00%> (+1.32%) ⬆️
...e/src/main/java/com/baidu/hugegraph/HugeGraph.java 65.30% <ø> (ø)
...in/java/com/baidu/hugegraph/StandardHugeGraph.java 67.50% <0.00%> (-0.98%) ⬇️
...a/com/baidu/hugegraph/auth/HugeGraphAuthProxy.java 58.35% <44.44%> (-0.45%) ⬇️
...c/main/java/com/baidu/hugegraph/task/HugeTask.java 72.00% <0.00%> (-0.31%) ⬇️
...du/hugegraph/backend/tx/GraphIndexTransaction.java 83.72% <0.00%> (-0.22%) ⬇️
.../baidu/hugegraph/backend/query/ConditionQuery.java 85.94% <0.00%> (+0.26%) ⬆️
...hugegraph/backend/serializer/BinarySerializer.java 85.44% <0.00%> (+0.31%) ⬆️
...va/com/baidu/hugegraph/task/ServerInfoManager.java 71.34% <0.00%> (+0.56%) ⬆️
... and 20 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 d213166...e8f5c1a. Read the comment docs.

@javeme javeme merged commit b6abf96 into master Dec 27, 2021
@javeme javeme deleted the fix-create-graph-with-auth branch December 27, 2021 06:22
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.

None yet

3 participants