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 query edges by multi labels + properties #1737

Merged
merged 3 commits into from
Jan 21, 2022

Conversation

javeme
Copy link
Contributor

@javeme javeme commented Jan 20, 2022

fix: #1735 #1736
Change-Id: I28e3a209074abaab8c8271775679d54845e6e785

fix: #1735 #1736
Change-Id: I28e3a209074abaab8c8271775679d54845e6e785
@codecov
Copy link

codecov bot commented Jan 20, 2022

Codecov Report

Merging #1737 (d082f2d) into master (e9cef74) will increase coverage by 0.64%.
The diff coverage is 73.52%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1737      +/-   ##
============================================
+ Coverage     66.46%   67.10%   +0.64%     
- Complexity      684      970     +286     
============================================
  Files           443      443              
  Lines         37678    37704      +26     
  Branches       5361     5370       +9     
============================================
+ Hits          25042    25303     +261     
+ Misses        10035     9796     -239     
- Partials       2601     2605       +4     
Impacted Files Coverage Δ
.../baidu/hugegraph/backend/query/ConditionQuery.java 85.14% <72.72%> (-0.84%) ⬇️
...u/hugegraph/traversal/optimize/HugeVertexStep.java 75.00% <100.00%> (ø)
...va/com/baidu/hugegraph/util/collection/IntSet.java 74.04% <0.00%> (-1.28%) ⬇️
...m/baidu/hugegraph/backend/tx/GraphTransaction.java 80.36% <0.00%> (+0.20%) ⬆️
...aidu/hugegraph/backend/store/mysql/MysqlTable.java 81.59% <0.00%> (+0.54%) ⬆️
...va/com/baidu/hugegraph/backend/cache/RamCache.java 73.03% <0.00%> (+0.56%) ⬆️
...in/java/com/baidu/hugegraph/api/graph/EdgeAPI.java 64.76% <0.00%> (+1.03%) ⬆️
...ain/java/com/baidu/hugegraph/task/TaskManager.java 74.82% <0.00%> (+1.43%) ⬆️
...va/com/baidu/hugegraph/task/ServerInfoManager.java 75.28% <0.00%> (+1.68%) ⬆️
...u/hugegraph/backend/store/mysql/MysqlSessions.java 69.06% <0.00%> (+2.54%) ⬆️
... and 8 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 e9cef74...d082f2d. Read the comment docs.

Change-Id: I24eec52bc98e456e2e968aae992187e97d725b33
@javeme javeme force-pushed the fix-query-edge-by-labels branch from 8cd4970 to 9832eb9 Compare January 20, 2022 14:07
Change-Id: Ie442cfb0355f36398422ae0616f4c5bce99cc79f
@javeme javeme force-pushed the fix-query-edge-by-labels branch from 9832eb9 to d082f2d Compare January 20, 2022 14:20
@javeme javeme merged commit cadeade into master Jan 21, 2022
@javeme javeme deleted the fix-query-edge-by-labels branch January 21, 2022 09:25
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.

[Bug] "hasLabel()" containing more than one labels cannot be queried after "outE"
3 participants