rocksdict/RocksDict

main branch no more using rocksdb v9 ?

raphaelauv opened this issue · 3 comments

Since the merge of this commit
128ef0f#diff-2e9d962a08321605940b5a657135052fbcef87b5e360662bb527c96d9a615542R14

the main branch is no more based on https://github.com/Congyuwang/rust-rocksdb but on rust-rocksdb

does it mean RocksDict is no more using rocksdb v9 ?

maybe a compatibility matrix in the readme of RocksDict and rocksdb versions would be great , wdyt ? thanks

No it is using rocksdb v9. The main branch has added some C-API to support Columns api, the PR of which has not yet been accepted by RocksDB.

thanks for your answer @Congyuwang , but I'm note sure to understand

this issue is still open rust-rocksdb/rust-rocksdb#884

Support for rocksdb v9 in Rocksdict:

Rocksdict uses v9 since this PR #119 and this release https://github.com/Congyuwang/RocksDict/releases/tag/v0.3.23.

Support for rocksdb v9 in rust-rocksdb:

Rocksdict uses a fork of rust-rocksdb, the main branch of which now supports rocksdb v9 since this PR: rust-rocksdb/rust-rocksdb#872.

Which rocksdb is currently used in the main branch?

However, currently Rocksdict main branch uses a fork of rocksdb: https://github.com/Congyuwang/rocksdb/tree/widecolumns-put-get-c-api because this PR facebook/rocksdb#12653 has not yet been merged.