Skip to content

Commit 8d90676

Browse files
committed
tracker: throttle empty probes
Signed-off-by: Pavel Kalinnikov <[email protected]>
1 parent 9dbaf80 commit 8d90676

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

tracker/progress.go

+1-6
Original file line numberDiff line numberDiff line change
@@ -165,12 +165,7 @@ func (pr *Progress) SentEntries(entries int, bytes uint64) {
165165
// consider this message being a probe, so that the flow is paused.
166166
pr.MsgAppFlowPaused = pr.Inflights.Full()
167167
case StateProbe:
168-
// TODO(pavelkalinnikov): this condition captures the previous behaviour,
169-
// but we should set MsgAppFlowPaused unconditionally for simplicity, because any
170-
// MsgApp in StateProbe is a probe, not only non-empty ones.
171-
if entries > 0 {
172-
pr.MsgAppFlowPaused = true
173-
}
168+
pr.MsgAppFlowPaused = true
174169
default:
175170
panic(fmt.Sprintf("sending append in unhandled state %s", pr.State))
176171
}

0 commit comments

Comments
 (0)