New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
repair: Introduce repair_partition_estimation_ratio config option #18634
Conversation
db/config.cc
Outdated
@@ -968,6 +968,8 @@ db::config::config(std::shared_ptr<db::extensions> exts) | |||
, enable_repair_based_node_ops(this, "enable_repair_based_node_ops", liveness::LiveUpdate, value_status::Used, true, "Set true to use enable repair based node operations instead of streaming based.") | |||
, allowed_repair_based_node_ops(this, "allowed_repair_based_node_ops", liveness::LiveUpdate, value_status::Used, "replace,removenode,rebuild,bootstrap,decommission", "A comma separated list of node operations which are allowed to enable repair based node operations. The operations can be bootstrap, replace, removenode, decommission and rebuild.") | |||
, enable_compacting_data_for_streaming_and_repair(this, "enable_compacting_data_for_streaming_and_repair", liveness::LiveUpdate, value_status::Used, true, "Enable the compacting reader, which compacts the data for streaming and repair (load'n'stream included) before sending it to, or synchronizing it with peers. Can reduce the amount of data to be processed by removing dead data, but adds CPU overhead.") | |||
, repair_partition_estimation_ratio(this, "repair_partition_estimation_ratio", liveness::LiveUpdate, value_status::Used, 0.1, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should have the word count
in the name, becuase currently it is unclear what quality of a partiiton we estimate here. I suggest: repair_partition_count_estimate_fraction
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Originally, I had a word number in the name which I dropped. The count looks good. Changed in v2.
In commit 642f9a1 (repair: Improve estimated_partitions to reduce memory usage), a 10% hard coded estimation ratio is used. This patch introduces a new config option to specify the estimation ratio of partitions written by repair out of the total partitions. It is set to 0.1 by default. Fixes scylladb#18615
025c380
to
340eae0
Compare
🟢 CI State: SUCCESS✅ - Build Build Details:
|
In commit 642f9a1 (repair: Improve estimated_partitions to reduce memory usage), a 10% hard coded estimation ratio is used.
This patch introduces a new config option to specify the estimation ratio of partitions written by repair out of the total partitions.
It is set to 0.1 by default.
Fixes #18615