Skip to content

Commit 6766271

Browse files
craig[bot]wenyihu6
andcommitted
Merge #152749
152749: kvserver: default PriorityInversionRequeue to true r=tbg a=wenyihu6 Previously, we introduced PriorityInversionRequeue to handle cases where a range is enqueued with a high-priority action but computes a lower-priority action at processing, re-adding the range with its updated priority. We plan to backport this setting soon. To allow some bake time, we enable it on true in master, while the backport will include only the previous PR with the default value set to false. Epic: none Release note: none Co-authored-by: wenyihu6 <[email protected]>
2 parents 65656d4 + e6f1746 commit 6766271

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

pkg/kv/kvserver/replicate_queue.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -106,13 +106,12 @@ var EnqueueProblemRangeInReplicateQueueInterval = settings.RegisterDurationSetti
106106
// PriorityInversionRequeue is a setting that controls whether to requeue
107107
// replicas when their priority at enqueue time and processing time is inverted
108108
// too much (e.g. dropping from a repair action to AllocatorConsiderRebalance).
109-
// TODO(wenyihu6): flip default to true after landing 152596 to bake
110109
var PriorityInversionRequeue = settings.RegisterBoolSetting(
111110
settings.SystemOnly,
112111
"kv.priority_inversion_requeue_replicate_queue.enabled",
113112
"whether the requeue replicas should requeue when enqueued for "+
114113
"repair action but ended up consider rebalancing during processing",
115-
false,
114+
true,
116115
)
117116

118117
// ReplicateQueueMaxSize is a setting that controls the max size of the

0 commit comments

Comments
 (0)