Eliminate redundant dial mutex causing unbounded connection queue contention

LINKIWI · LINKIWI · commit fedd45f443c6 · 2024-08-10T07:12:48.000-07:00
diff --git a/redis.go b/redis.go
@@ -176,8 +176,6 @@ func (hs *hooksMixin) withProcessPipelineHook(
 }
 
 func (hs *hooksMixin) dialHook(ctx context.Context, network, addr string) (net.Conn, error) {
-	hs.hooksMu.Lock()
-	defer hs.hooksMu.Unlock()
 	return hs.current.dial(ctx, network, addr)
 }
 

Original file line number	Diff line number	Diff line change
`@@ -176,8 +176,6 @@ func (hs *hooksMixin) withProcessPipelineHook(`
`176`	`176`	`}`
`177`	`177`
`178`	`178`	`func (hs *hooksMixin) dialHook(ctx context.Context, network, addr string) (net.Conn, error) {`
`179`		`- hs.hooksMu.Lock()`
`180`		`- defer hs.hooksMu.Unlock()`
`181`	`179`	`return hs.current.dial(ctx, network, addr)`
`182`	`180`	`}`
`183`	`181`