tarantool / go-storage / 24719176424

90%

Build Type

push

github

Committed by bigbes

Commit Message

fix: prevent goroutine leak in storage.Watch

The testrace target was timing out in the integrity package
after 600 seconds. The root cause was 207 goroutines blocked
on `chan receive (nil chan)` at storage.go:116.

The cleanup goroutine in Watch() was blocking forever on
`<-ctx.Done()` when ctx.Done() returns nil (e.g., with
context.Background()). With testrace running -count=100,
these leaked goroutines accumulated and caused timeouts.

The forwarding goroutine now checks for context cancellation
and exits properly. The cleanup goroutine waits on either
ctx.Done() or a new done channel, ensuring cleanup runs and
goroutines terminate in all scenarios.

Coverage Stats

17 of 19 new or added lines in 1 file covered. (89.47%)

9 existing lines in 1 file now uncovered.

1849 of 1968 relevant lines covered (93.95%)

27.63 hits per line