auth fix: creator is null in toString() called by consistent-hash LB when verifying permission by xuliguov5 · Pull Request #1385 · apache/hugegraph
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 583
Merged
Conversation
Copy link
Contributor
xuliguov5
commented
Mar 10, 2021
xuliguov5
commented
…reator of auth element is null by tostring method.
…reator of auth element is null by tostring method.
Copy link
Codecov Report
Merging #1385 (a36d75f) into master (8bda451) will increase coverage by
0.00%.
The diff coverage is0.00%.
@@ Coverage Diff @@ ## master #1385 +/- ## ========================================= Coverage 62.36% 62.36% Complexity 5827 5827 ========================================= Files 385 385 Lines 31955 31955 Branches 4460 4460 ========================================= + Hits 19929 19930 +1 Misses 10003 10003 + Partials 2023 2022 -1
| Impacted Files | Coverage Δ | Complexity Δ | |
|---|---|---|---|
| ...a/com/baidu/hugegraph/auth/HugeGraphAuthProxy.java | 3.98% <0.00%> (ø) |
0.00 <0.00> (ø) |
|
| ...va/com/baidu/hugegraph/task/ServerInfoManager.java | 70.78% <0.00%> (+0.56%) |
51.00% <0.00%> (+1.00%) |
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 8bda451...a36d75f. Read the comment docs.
javeme
changed the title
Consistent hash load balancing is done according to the method, the c…
auth fix: creator is null in toString() called by consistent-hash LB when verifying permission
javeme approved these changes Mar 10, 2021
coderzc approved these changes Mar 10, 2021
Linary approved these changes Mar 10, 2021
Linary
merged commit
17ac195
into
master
Linary
deleted the
auth-element-tostring-asmap
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment