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

unify action of special-symbol in primary-key and sort-key #1684

Merged
merged 2 commits into from
Dec 13, 2021

Conversation

javeme
Copy link
Contributor

@javeme javeme commented Dec 9, 2021

related: #1670
Change-Id: I1cb380396588dbe6aea3d4a8b68a9cdf675bebd7

@codecov
Copy link

codecov bot commented Dec 9, 2021

Codecov Report

Merging #1684 (bdc59e1) into master (f6a492c) will increase coverage by 0.33%.
The diff coverage is 76.47%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1684      +/-   ##
============================================
+ Coverage     66.74%   67.07%   +0.33%     
- Complexity     7052     7078      +26     
============================================
  Files           420      420              
  Lines         34668    34678      +10     
  Branches       4805     4808       +3     
============================================
+ Hits          23139    23262     +123     
+ Misses         9189     9081     -108     
+ Partials       2340     2335       -5     
Impacted Files Coverage Δ
...n/java/com/baidu/hugegraph/schema/PropertyKey.java 76.97% <50.00%> (-0.56%) ⬇️
...n/java/com/baidu/hugegraph/structure/HugeEdge.java 76.23% <75.00%> (-0.27%) ⬇️
...java/com/baidu/hugegraph/structure/HugeVertex.java 78.79% <100.00%> (+0.22%) ⬆️
...va/com/baidu/hugegraph/auth/HugeAuthenticator.java 34.78% <0.00%> (-5.44%) ⬇️
...in/java/com/baidu/hugegraph/auth/HugeResource.java 77.92% <0.00%> (-1.30%) ⬇️
.../java/com/baidu/hugegraph/auth/RolePermission.java 90.58% <0.00%> (-1.18%) ⬇️
...va/com/baidu/hugegraph/backend/cache/RamCache.java 72.47% <0.00%> (-0.57%) ⬇️
...m/baidu/hugegraph/backend/tx/GraphTransaction.java 80.16% <0.00%> (+0.30%) ⬆️
...a/com/baidu/hugegraph/backend/query/Condition.java 79.78% <0.00%> (+0.36%) ⬆️
...egraph/backend/store/cassandra/CassandraStore.java 72.64% <0.00%> (+0.85%) ⬆️
... and 6 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 f6a492c...bdc59e1. Read the comment docs.

Change-Id: I1cb380396588dbe6aea3d4a8b68a9cdf675bebd7
Change-Id: Ib8179d878b4311f72268f80e39f4aea48bba68b7
@javeme javeme force-pushed the special-char-in-primarykey-sortkey branch from f6d6427 to bdc59e1 Compare December 9, 2021 19:55
@javeme javeme merged commit cd22604 into master Dec 13, 2021
@javeme javeme deleted the special-char-in-primarykey-sortkey branch December 13, 2021 06:02
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