Skip to content

Commit

Permalink
Merge #104741
Browse files Browse the repository at this point in the history
104741: kvflowcontrol: enable by default r=irfansharif a=irfansharif

Enable kvadmission.flow_control.enabled by default. We didn't observe noticeable performance regressions while it was disabled (single weekly run, three nightly runs). There was some minimal fallout that was since fixed (#104699). We expect performance regressions now that this commit enables it by default, and expect more fallout. We'll handle these as part of #104154.

Release note: None

Co-authored-by: irfan sharif <irfanmahmoudsharif@gmail.com>
  • Loading branch information
craig[bot] and irfansharif committed Jun 12, 2023
2 parents 3d4db0a + d75e8bc commit 8814ce5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/kv/kvserver/kvflowcontrol/kvflowcontrol.go
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ var Enabled = settings.RegisterBoolSetting(
settings.SystemOnly,
"kvadmission.flow_control.enabled",
"determines whether we use flow control for replication traffic in KV",
false,
true,
)

// Mode determines the 'mode' of flow control we use for replication traffic in
Expand Down

0 comments on commit 8814ce5

Please sign in to comment.