Skip to content

Conversation

@lhotari
Copy link
Member

@lhotari lhotari commented Mar 5, 2025

Motivation

RocksDB config for the file based configuration alternative has been missing from bk_server.conf

Changes

  • Add comments and missing keys to bk_server.conf

This depends on #4559 so that dbStorage_rocksDB_format_version would apply to all RocksDB databases.

Copy link
Member

@StevenLuMT StevenLuMT left a comment

Choose a reason for hiding this comment

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

Good jobs

@StevenLuMT StevenLuMT merged commit 64abf31 into apache:master Apr 1, 2025
23 checks passed
lhotari added a commit that referenced this pull request Apr 16, 2025
lhotari added a commit that referenced this pull request Apr 16, 2025
lhotari added a commit that referenced this pull request Apr 17, 2025
priyanshu-ctds pushed a commit to datastax/bookkeeper that referenced this pull request Jul 11, 2025
(cherry picked from commit 64abf31)
(cherry picked from commit e06f3a5)
sandeep-ctds pushed a commit to datastax/bookkeeper that referenced this pull request Jul 22, 2025
(cherry picked from commit 64abf31)
(cherry picked from commit e06f3a5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants