Skip to content

Commit 332301e

Browse files
committed
rangefeed: add assertion that OnError is called on disconnected registration
Epic: none Release note: None
1 parent a806cde commit 332301e

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

pkg/kv/kvserver/rangefeed/stream_manager.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,8 +121,8 @@ func (sm *StreamManager) OnError(streamID int64) {
121121
func() {
122122
sm.streams.Lock()
123123
defer sm.streams.Unlock()
124-
if _, ok := sm.streams.m[streamID]; ok {
125-
// TODO(ssd): We should be able to assert we are disconnected here.
124+
if d, ok := sm.streams.m[streamID]; ok {
125+
assertTrue(d.IsDisconnected(), "OnError called on connected registration")
126126
delete(sm.streams.m, streamID)
127127
sm.metrics.ActiveMuxRangeFeed.Dec(1)
128128
}

0 commit comments

Comments
 (0)