Skip to content

kvserver: bump snapshot rates from 8 MB/s to 32 MB/s#71814

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
irfansharif:211021.bump-snapshot-rates
Oct 21, 2021
Merged

kvserver: bump snapshot rates from 8 MB/s to 32 MB/s#71814
craig[bot] merged 1 commit intocockroachdb:masterfrom
irfansharif:211021.bump-snapshot-rates

Conversation

@irfansharif
Copy link
Copy Markdown
Contributor

For years we've found these rates to be too conservative, and the
bottleneck during recovery/replication. We have customers running at
much higher rates (128 MB/s) for production clusters. Let's bump these
default and let it bake over the release to see what breaks.

Release note: The default snapshot recovery/rebalance rates
kv.snapshot_rebalance.max_rate and kv.snapshot_recovery.max_rate
were bumped from 8MB/s to 32MB/s, production experience has taught us
that earlier values were too conservative. Users might observer higher
network utilization during rebalancing/recovery in service of
rebalancing/recovering faster (for the latter, possibly reducing the
MTTF). If the extra utilization is undesirable, users can manually
revert these rates back to their original settings of 8 MB/s.

@irfansharif irfansharif requested review from nvb and tbg October 21, 2021 15:20
@irfansharif irfansharif requested a review from a team as a code owner October 21, 2021 15:20
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@irfansharif irfansharif force-pushed the 211021.bump-snapshot-rates branch from eec0d75 to e559012 Compare October 21, 2021 18:18
For years we've found these rates to be too conservative, and the
bottleneck during recovery/replication. We have customers running at
much higher rates (128 MB/s) for production clusters. Let's bump these
default and let it bake over the release to see what breaks.

Release note: The default snapshot recovery/rebalance rates
`kv.snapshot_rebalance.max_rate` and `kv.snapshot_recovery.max_rate`
were bumped from 8MB/s to 32MB/s, production experience has taught us
that earlier values were too conservative. Users might observer higher
network utilization during rebalancing/recovery in service of
rebalancing/recovering faster (for the latter, possibly reducing the
MTTF). If the extra utilization is undesirable, users can manually
revert these rates back to their original settings of 8 MB/s.
@irfansharif irfansharif force-pushed the 211021.bump-snapshot-rates branch from e559012 to c6f21af Compare October 21, 2021 18:31
@irfansharif
Copy link
Copy Markdown
Contributor Author

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Oct 21, 2021

Build failed (retrying...):

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Oct 21, 2021

Build succeeded:

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.

3 participants