go - redigo 连接池 - 为什么在删除陈旧连接时释放锁

Question

Redigo 是 redis 数据库的 golang 客户端。它使用 struct Pool 来维护连接池。该结构持有一个互斥锁，用于并行放置和获取连接的应用程序。

type Pool struct {
    // ...
    IdleTimeout time.Duration
    mu     sync.Mutex
    // Stack of idleConn with most recently used at the front.
    idle list.List
}

在其get 方法中，连接池首先移除过时(空闲超时)连接。当找到失效连接时，池将其弹出，释放锁，然后关闭连接，尝试再次获取锁。

func (p *Pool) get() (Conn, error) {
    p.mu.Lock()

    // Prune stale connections.

    if timeout := p.IdleTimeout; timeout > 0 {
        for i, n := 0, p.idle.Len(); i < n; i++ {
            e := p.idle.Back()
            if e == nil {
                break
            }
            ic := e.Value.(idleConn)
                if ic.t.Add(timeout).After(nowFunc()) {
                    break
                }
            p.idle.Remove(e)
            p.release()
            // Why does pool unlock and try to acquire lock again?
            p.mu.Unlock()
            // Close this stale connection.
            ic.c.Close()
            p.mu.Lock()
        }
    }

为什么池解锁并尝试再次获取锁，而不是在函数返回之前解锁？我想关闭一个连接可能会花费很多时间，这会减慢等待这个互斥锁的其他 goroutine 的速度。

这是全部 Pool get method

Answer 1

关闭连接可能会花费很多时间，这会减慢等待此互斥量的其他 goroutine。正如@Cerise Limón 所说 - 这次锁定所有池的使用似乎是不明智的。

解锁互斥锁后，其中一个等待的 goroutine 获得互斥锁。虽然 get 方法的 goroutine 仍然需要移除过时的连接，但是 put 方法的 goroutine 可以尽快将连接放入池中并继续做其他工作。