Skip to content

WIP - max_background_remote_compactions#14378

Draft
jaykorean wants to merge 1 commit intofacebook:mainfrom
jaykorean:max_background_remote_compactions
Draft

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

Conversation

@jaykorean
Copy link
Contributor

No description provided.

@meta-cla meta-cla bot added the CLA Signed label Feb 24, 2026
@github-actions
Copy link

✅ clang-tidy: No findings on changed lines

Completed in 266.8s.

// Default: 0
//
// Dynamically changeable through SetDBOptions() API.
int max_background_remote_compactions = 0;
Copy link
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.

Copy link
Contributor

@hx235 hx235 left a comment

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants