Skip to content

Commit e6f1746

Browse files
committed
kvserver: default PriorityInversionRequeue to true
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.
1 parent 205e55a commit e6f1746

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)