-
Notifications
You must be signed in to change notification settings - Fork 522
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
bump up rocksdb version to 6.22 #1742
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1742 +/- ##
============================================
+ Coverage 66.75% 71.12% +4.37%
- Complexity 728 964 +236
============================================
Files 451 452 +1
Lines 38207 38520 +313
Branches 5407 5421 +14
============================================
+ Hits 25505 27399 +1894
+ Misses 10185 8459 -1726
- Partials 2517 2662 +145
Continue to review full report at Codecov.
|
hugegraph-rocksdb/pom.xml
Outdated
@@ -20,7 +20,7 @@ | |||
<dependency> | |||
<groupId>org.rocksdb</groupId> | |||
<artifactId>rocksdbjni</artifactId> | |||
<version>6.10.2</version> | |||
<version>6.22.1.1</version> |
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.
TODO: bump up to 6.30 when it's released (now 6.27.3 available, but not supports auto prefix mode and memtable whole key filtering)
rangeInt(0, Integer.MAX_VALUE), | ||
0 | ||
); | ||
|
||
public static final ConfigOption<Double> MEMTABLE_PREFIX_BLOOM_SIZE_RATIO = | ||
new ConfigOption<>( | ||
"rocksdb.memtable_prefix_bloom_size_ratio", |
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.
rename to memtable_bloom_size_ratio
: https://github.com/facebook/rocksdb/pull/9453/files#diff-cde52d1fcbcce2bc6aae27838f1d3e7e9e469ccad8aaf8f2695f939e279d7501R369
Due to the lack of activity, the current pr is marked as stale and will be closed after 180 days, any update will remove the stale label |
Due to the lack of activity, the current pr is marked as stale and will be closed after 180 days, any update will remove the stale label |
Change-Id: I357f0c2ced0c63d1e357b52f7397fb936255b411
Change-Id: I3827d6229ce877fd251d1b0b76219318b53bb542
Change-Id: I014c5c00ee9da3103b5e31d0df52c2188cfd163e
Change-Id: I2d9d4e46f1307142e24344e9345ed957b3c8a6a3
5c09ecb
to
2f40afc
Compare
Change-Id: I9658101b5c855a57a824570c0925d1054c9d6f17
Change-Id: Id075edf08b95510fa732ace6da05a38f68088c2f
Change-Id: I357f0c2ced0c63d1e357b52f7397fb936255b411