Skip to content

WIP - max_background_remote_compactions#14378

Draft
jaykorean wants to merge 1 commit into
facebook:mainfrom
jaykorean:max_background_remote_compactions
Draft

WIP - max_background_remote_compactions#14378
jaykorean wants to merge 1 commit into
facebook:mainfrom
jaykorean:max_background_remote_compactions

Conversation

@jaykorean

Copy link
Copy Markdown
Contributor

No description provided.

@meta-cla meta-cla Bot added the CLA Signed label Feb 24, 2026
@github-actions

Copy link
Copy Markdown

✅ clang-tidy: No findings on changed lines

Completed in 266.8s.

Comment thread include/rocksdb/options.h
// Default: 0
//
// Dynamically changeable through SetDBOptions() API.
int max_background_remote_compactions = 0;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

See max_background_compactions/flush above for its DEPRECATED note. I was going to remove them in the near future. RocksDB has been on a journey since then or even earlier than that to minimize exposing low-level configuration. We can talk more about the motivation behind this PR and see what are the other approaches if you want.

@hx235 hx235 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If this is becoming an official work, let's discuss more about #14378 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

2 participants