Merge pull request #636 from go-redis/fix/pool-stats-alias

PoolStats is an alias for pool.Stats

Author: vmihailenco

options.go

@@ -198,14 +198,3 @@ func newConnPool(opt *Options) *pool.ConnPool {
 		IdleCheckFrequency: opt.IdleCheckFrequency,
 	})
 }
-
-// PoolStats contains pool state information and accumulated stats.
-type PoolStats struct {
-	Requests uint32 // number of times a connection was requested by the pool
-	Hits     uint32 // number of times free connection was found in the pool
-	Timeouts uint32 // number of times a wait timeout occurred
-
-	TotalConns uint32 // number of total connections in the pool
-	FreeConns  uint32 // number of free connections in the pool
-	StaleConns uint32 // number of stale connections removed from the pool
-}

pool_test.go

@@ -125,6 +125,7 @@ var _ = Describe("pool", func() {
 			Timeouts:   0,
 			TotalConns: 1,
 			FreeConns:  1,
+			StaleConns: 0,
 		}))
 
 		time.Sleep(2 * time.Second)
@@ -136,6 +137,7 @@ var _ = Describe("pool", func() {
 			Timeouts:   0,
 			TotalConns: 0,
 			FreeConns:  0,
+			StaleConns: 1,
 		}))
 	})
 })

redis.go

@@ -352,17 +352,12 @@ func (c *Client) Options() *Options {
 	return c.opt
 }
 
+type PoolStats pool.Stats
+
 // PoolStats returns connection pool stats.
 func (c *Client) PoolStats() *PoolStats {
-	s := c.connPool.Stats()
-	return &PoolStats{
-		Requests: s.Requests,
-		Hits:     s.Hits,
-		Timeouts: s.Timeouts,
-
-		TotalConns: s.TotalConns,
-		FreeConns:  s.FreeConns,
-	}
+	stats := c.connPool.Stats()
+	return (*PoolStats)(stats)
 }
 
 func (c *Client) Pipelined(fn func(Pipeliner) error) ([]Cmder, error) {