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

fixed repeat lock in LocksTable #566

Merged
merged 2 commits into from
Jun 14, 2019
Merged

fixed repeat lock in LocksTable #566

merged 2 commits into from
Jun 14, 2019

Conversation

zhoney
Copy link
Contributor

@zhoney zhoney commented Jun 13, 2019

fixed: #564

Change-Id: I026e4a6739c7f526c421cbb5c90b48e5ff8de066

fixed: #564

Change-Id: I026e4a6739c7f526c421cbb5c90b48e5ff8de066
@codecov
Copy link

codecov bot commented Jun 13, 2019

Codecov Report

Merging #566 into master will decrease coverage by 0.05%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #566      +/-   ##
============================================
- Coverage     70.44%   70.39%   -0.06%     
- Complexity     3444     3450       +6     
============================================
  Files           216      216              
  Lines         16487    16535      +48     
  Branches       2352     2355       +3     
============================================
+ Hits          11615    11639      +24     
- Misses         3629     3653      +24     
  Partials       1243     1243
Impacted Files Coverage Δ Complexity Δ
...c/main/java/com/baidu/hugegraph/util/LockUtil.java 79.8% <100%> (+1.37%) 10 <0> (ø) ⬇️
.../baidu/hugegraph/backend/store/BackendSession.java 82.35% <0%> (-7.65%) 8% <0%> (+2%)
...u/hugegraph/backend/store/mysql/MysqlSessions.java 69.27% <0%> (-1.59%) 20% <0%> (ø)
...du/hugegraph/backend/store/BackendSessionPool.java 82.14% <0%> (-0.47%) 15% <0%> (+2%)
...raph/backend/store/rocksdb/RocksDBStdSessions.java 69.65% <0%> (-0.19%) 24% <0%> (ø)
...h/backend/store/rocksdbsst/RocksDBSstSessions.java 0% <0%> (ø) 0% <0%> (ø) ⬇️
...gegraph/backend/store/rocksdb/RocksDBSessions.java 100% <0%> (ø) 1% <0%> (ø) ⬇️
...java/com/baidu/hugegraph/dist/HugeGraphServer.java 0% <0%> (ø) 0% <0%> (ø) ⬇️
...n/java/com/baidu/hugegraph/config/CoreOptions.java 98.64% <0%> (+0.05%) 2% <0%> (ø) ⬇️
...e/src/main/java/com/baidu/hugegraph/HugeGraph.java 72.87% <0%> (+0.08%) 64% <0%> (ø) ⬇️
... and 5 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 88f249a...de45685. Read the comment docs.

Change-Id: I7cb72c56a70f4a5f9cd322d67dedacc4dbcde16a
@Linary Linary merged commit b9763c6 into master Jun 14, 2019
@Linary Linary deleted the max-lock branch June 14, 2019 09:58
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.

Maximum lock count exceeded when add edge
3 participants