-
Notifications
You must be signed in to change notification settings - Fork 504
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 rocksdb instance not close properly #1264
Conversation
* The open-pool opened session is always exist Change-Id: I30504565155848659a818fbc1b57999547f0502f
35a97a2
to
74a4516
Compare
@@ -217,6 +218,16 @@ private static void waitOpenFinish(List<Future<?>> futures, | |||
if (openPool.isShutdown()) { | |||
return; | |||
} | |||
|
|||
this.sessions.session(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add comments for why
Change-Id: I5ce6d06d340d8ded8096b1f016e6a1d7f7edeea8
Codecov Report
@@ Coverage Diff @@
## master #1264 +/- ##
============================================
+ Coverage 62.39% 62.43% +0.03%
- Complexity 5758 5766 +8
============================================
Files 374 374
Lines 31295 31307 +12
Branches 4398 4398
============================================
+ Hits 19528 19546 +18
+ Misses 9779 9778 -1
+ Partials 1988 1983 -5
Continue to review full report at Codecov.
|
hugegraph-rocksdb/src/main/java/com/baidu/hugegraph/backend/store/rocksdb/RocksDBStore.java
Outdated
Show resolved
Hide resolved
this.sessions.session(); | ||
try { | ||
Consumers.executeOncePerThread(openPool, OPEN_POOL_THREADS, | ||
() -> this.sessions.closeSession()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it ok if there are multi disks
Change-Id: I184c428b09420c435121f1076e830a992ad171fd
Change-Id: I19e3d33fef1aa7db9c60bc38c3942dc94f979faf
* The open-pool opened session is always exist * Fix other disk sessions not close Change-Id: I30504565155848659a818fbc1b57999547f0502f
Change-Id: I30504565155848659a818fbc1b57999547f0502f