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

Support offheap cache #846

Merged
merged 8 commits into from
Apr 7, 2020
Merged

Support offheap cache #846

merged 8 commits into from
Apr 7, 2020

Conversation

javeme
Copy link
Contributor

@javeme javeme commented Feb 2, 2020

implememnt #844
Change-Id: Ib0170b5d6b442231f77dba29e4b2d2270225c935

@codecov
Copy link

codecov bot commented Feb 4, 2020

Codecov Report

Merging #846 into master will increase coverage by 2.92%.
The diff coverage is 79.80%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #846      +/-   ##
============================================
+ Coverage     69.13%   72.05%   +2.92%     
- Complexity     4503     4674     +171     
============================================
  Files           295      298       +3     
  Lines         22045    22293     +248     
  Branches       3121     3153      +32     
============================================
+ Hits          15240    16063     +823     
+ Misses         5308     4668     -640     
- Partials       1497     1562      +65     
Impacted Files Coverage Δ Complexity Δ
...du/hugegraph/backend/cache/CachedBackendStore.java 14.28% <0.00%> (+4.76%) 0.00 <0.00> (ø)
...hugegraph/backend/serializer/BinarySerializer.java 83.24% <0.00%> (-0.06%) 107.00 <0.00> (+3.00) ⬇️
...m/baidu/hugegraph/backend/cache/AbstractCache.java 70.29% <70.29%> (ø) 28.00 <28.00> (?)
...va/com/baidu/hugegraph/backend/cache/RamCache.java 71.83% <71.42%> (-1.89%) 27.00 <9.00> (-24.00)
...om/baidu/hugegraph/backend/cache/OffheapCache.java 76.54% <76.54%> (ø) 16.00 <16.00> (?)
.../com/baidu/hugegraph/backend/cache/LevelCache.java 91.89% <91.89%> (ø) 19.00 <19.00> (?)
...in/java/com/baidu/hugegraph/core/GraphManager.java 55.22% <100.00%> (+4.09%) 0.00 <0.00> (ø)
...om/baidu/hugegraph/backend/cache/CacheManager.java 88.23% <100.00%> (+6.98%) 12.00 <5.00> (+4.00)
...ugegraph/backend/cache/CachedGraphTransaction.java 95.45% <100.00%> (+0.08%) 32.00 <1.00> (ø)
...gegraph/backend/cache/CachedSchemaTransaction.java 96.61% <100.00%> (ø) 28.00 <0.00> (ø)
... and 24 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 a24f7ec...1c98c7c. Read the comment docs.

.eviction(Eviction.LRU);
}

// private OHCacheBuilder<byte[], byte[]> builder() {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

delete it?

implememnt #844
Change-Id: Ib0170b5d6b442231f77dba29e4b2d2270225c935
Change-Id: I3e5bc32305309c9862a9d548b3302a5b47cafabd
Change-Id: Ic6b9fc056349f5a48f5de10a25bf08c7902c5832
Change-Id: I74f442744fd39bcf8b25453c564b808ea368503b
Change-Id: I523bc84ae33bd74b85bd03eb8fc7041f25e2e656
Change-Id: If8dad4db400d06df7616f1711a403d4c07f82d7e
Change-Id: I0ceafd4a749ca0981344930408980ca7a8c353af
Change-Id: I0af0c6228e22e934efe5ed62986bd446dc0c629c
@zhoney zhoney merged commit 95b1853 into master Apr 7, 2020
@zhoney zhoney deleted the cache-offheap branch April 7, 2020 12:34
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