Skip to content

Upgrade rocksdb dependency to 0.24#23

Open
citizen-stig wants to merge 9 commits intomainfrom
nikolai/upgrade-rocksb-to-0-23
Open

Upgrade rocksdb dependency to 0.24#23
citizen-stig wants to merge 9 commits intomainfrom
nikolai/upgrade-rocksb-to-0-23

Conversation

@citizen-stig
Copy link
Copy Markdown
Member

@citizen-stig citizen-stig commented Jun 5, 2025

Version mapping (Rust → C++)

rocksdb 0.22.0 bundled RocksDB 8.6.7 (8.3.2 → 8.5.3 → 8.6.7 during that release). GitHub](https://github.com/rust-rocksdb/rust-rocksdb/releases/tag/v0.22.0)

rocksdb 0.24.0 ends up on RocksDB 10.4.2 GitHub

Deprecations / behavior changes surfaced by the binding

  • DBOptions::set_ignore_range_deletions deprecated (RocksDB 10.2.1). Remove usages.
  • Options::set_skip_checking_sst_file_sizes_on_db_open deprecated. Remove usages.

Upstream C++ RocksDB changes that may affect

The big one is the 9.x line:

RocksDB 9.0.0 made format_version=6 the default in BlockBasedTableOptions for stronger integrity checks. SSTs written with this default are unreadable by RocksDB < 8.6.0. If you need downgrade/read-compat with older binaries, override format_version < 6 explicitly when writing.
New Releases

0.23 brought the big C++ 9.0 jump; added options like set_avoid_unnecessary_blocking_io, auto-tuned rate limiter toggle, blob cache enablement, per-CF TTL, LRU cache options, Env::from_raw, and more.


Open with Devin

@citizen-stig citizen-stig marked this pull request as ready for review June 19, 2025 12:15
@citizen-stig citizen-stig changed the title Upgrade rocksdb dependency to 0.23 Upgrade rocksdb dependency to 0.24 Aug 13, 2025
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

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.

1 participant