Skip to content

Commit 943240d

Browse files
Shorter comment
1 parent c7eb618 commit 943240d

File tree

1 file changed

+5
-11
lines changed

1 file changed

+5
-11
lines changed

balancer/rls/control_channel.go

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -208,17 +208,11 @@ func (cc *controlChannel) monitorConnectivityState() {
208208
seenTransientFailure = true
209209
}
210210

211-
// Only reset backoff when recovering from TRANSIENT_FAILURE to READY.
212-
// When the control channel enters TRANSIENT_FAILURE, it indicates the RLS
213-
// server is unreachable or experiencing issues. When it then transitions to
214-
// READY, we reset the backoff state in all cache entries to allow pending
215-
// RPCs to proceed immediately, rather than waiting for their individual
216-
// backoff timers to expire.
217-
//
218-
// We skip resetting backoff for benign state transitions like READY → IDLE
219-
// → READY (which occur during normal operation due to connection idleness)
220-
// because these don't represent actual failures that would justify clearing
221-
// the backoff state.
211+
// Only reset backoff when transitioning from TRANSIENT_FAILURE to READY.
212+
// This indicates the RLS server has recovered from being unreachable, so
213+
// we reset backoff state in all cache entries to allow pending RPCs to
214+
// proceed immediately. We skip benign transitions like READY → IDLE → READY
215+
// since those don't represent actual failures.
222216
if s == connectivity.Ready {
223217
if seenTransientFailure {
224218
cc.logger.Infof("Control channel back to READY after TRANSIENT_FAILURE")

0 commit comments

Comments
 (0)