Merge pull request #460 from pengfei-xue/develop

use connection pool for redis, support auto connection

cache/redis.go

@@ -3,7 +3,7 @@ package cache
 import (
 	"encoding/json"
 	"errors"
-	"io"
+	"time"
 
 	"github.com/beego/redigo/redis"
 )
@@ -15,7 +15,7 @@ var (
 
 // Redis cache adapter.
 type RedisCache struct {
-	c        redis.Conn
+	p        *redis.Pool // redis connection pool
 	conninfo string
 	key      string
 }
@@ -25,23 +25,17 @@ func NewRedisCache() *RedisCache {
 	return &RedisCache{key: DefaultKey}
 }
 
+// actually do the redis cmds
+func (rc *RedisCache) do(commandName string, args ...interface{}) (reply interface{}, err error) {
+	c := rc.p.Get()
+	defer c.Close()
+
+	return c.Do(commandName, args...)
+}
+
 // Get cache from redis.
 func (rc *RedisCache) Get(key string) interface{} {
-	if rc.c == nil {
+	v, err := rc.do("HGET", rc.key, key)
-		var err error
-		rc.c, err = rc.connectInit()
-		if err != nil {
-			return nil
-		}
-	}
-
-	v, err := rc.c.Do("HGET", rc.key, key)
-	// write to closed socket, reset rc.c to nil
-	if err == io.EOF {
-		rc.c = nil
-		return nil
-	}
-
 	if err != nil {
 		return nil
 	}
@@ -52,61 +46,19 @@ func (rc *RedisCache) Get(key string) interface{} {
 // put cache to redis.
 // timeout is ignored.
 func (rc *RedisCache) Put(key string, val interface{}, timeout int64) error {
-	if rc.c == nil {
+	_, err := rc.do("HSET", rc.key, key, val)
-		var err error
-		rc.c, err = rc.connectInit()
-		if err != nil {
-			return err
-		}
-	}
-
-	_, err := rc.c.Do("HSET", rc.key, key, val)
-	// write to closed socket, reset rc.c to nil
-	if err == io.EOF {
-		rc.c = nil
-		return err
-	}
-
 	return err
 }
 
 // delete cache in redis.
 func (rc *RedisCache) Delete(key string) error {
-	if rc.c == nil {
+	_, err := rc.do("HDEL", rc.key, key)
-		var err error
-		rc.c, err = rc.connectInit()
-		if err != nil {
-			return err
-		}
-	}
-
-	_, err := rc.c.Do("HDEL", rc.key, key)
-	// write to closed socket, reset rc.c to nil
-	if err == io.EOF {
-		rc.c = nil
-		return err
-	}
-
 	return err
 }
 
 // check cache exist in redis.
 func (rc *RedisCache) IsExist(key string) bool {
-	if rc.c == nil {
+	v, err := redis.Bool(rc.do("HEXISTS", rc.key, key))
-		var err error
-		rc.c, err = rc.connectInit()
-		if err != nil {
-			return false
-		}
-	}
-
-	v, err := redis.Bool(rc.c.Do("HEXISTS", rc.key, key))
-	// write to closed socket, reset rc.c to nil
-	if err == io.EOF {
-		rc.c = nil
-		return false
-	}
-
 	if err != nil {
 		return false
 	}
@@ -116,59 +68,19 @@ func (rc *RedisCache) IsExist(key string) bool {
 
 // increase counter in redis.
 func (rc *RedisCache) Incr(key string) error {
-	if rc.c == nil {
+	_, err := redis.Bool(rc.do("HINCRBY", rc.key, key, 1))
-		var err error
-		rc.c, err = rc.connectInit()
-		if err != nil {
-			return err
-		}
-	}
-
-	_, err := redis.Bool(rc.c.Do("HINCRBY", rc.key, key, 1))
-	// write to closed socket
-	if err == io.EOF {
-		rc.c = nil
-	}
-
 	return err
 }
 
 // decrease counter in redis.
 func (rc *RedisCache) Decr(key string) error {
-	if rc.c == nil {
+	_, err := redis.Bool(rc.do("HINCRBY", rc.key, key, -1))
-		var err error
-		rc.c, err = rc.connectInit()
-		if err != nil {
-			return err
-		}
-	}
-
-	_, err := redis.Bool(rc.c.Do("HINCRBY", rc.key, key, -1))
-
-	// write to closed socket
-	if err == io.EOF {
-		rc.c = nil
-	}
-
 	return err
 }
 
 // clean all cache in redis. delete this redis collection.
 func (rc *RedisCache) ClearAll() error {
-	if rc.c == nil {
+	_, err := rc.do("DEL", rc.key)
-		var err error
-		rc.c, err = rc.connectInit()
-		if err != nil {
-			return err
-		}
-	}
-
-	_, err := rc.c.Do("DEL", rc.key)
-	// write to closed socket
-	if err == io.EOF {
-		rc.c = nil
-	}
-
 	return err
 }
 
@@ -179,32 +91,42 @@ func (rc *RedisCache) ClearAll() error {
 func (rc *RedisCache) StartAndGC(config string) error {
 	var cf map[string]string
 	json.Unmarshal([]byte(config), &cf)
+
 	if _, ok := cf["key"]; !ok {
 		cf["key"] = DefaultKey
 	}
+
 	if _, ok := cf["conn"]; !ok {
 		return errors.New("config has no conn key")
 	}
+
 	rc.key = cf["key"]
 	rc.conninfo = cf["conn"]
-	var err error
+	rc.connectInit()
-	rc.c, err = rc.connectInit()
+
-	if err != nil {
+	c := rc.p.Get()
+	defer c.Close()
+	if err := c.Err(); err != nil {
 		return err
 	}
-	if rc.c == nil {
+
-		return errors.New("dial tcp conn error")
-	}
 	return nil
 }
 
 // connect to redis.
-func (rc *RedisCache) connectInit() (redis.Conn, error) {
+func (rc *RedisCache) connectInit() {
+	// initialize a new pool
+	rc.p = &redis.Pool{
+		MaxIdle:     3,
+		IdleTimeout: 180 * time.Second,
+		Dial: func() (redis.Conn, error) {
 			c, err := redis.Dial("tcp", rc.conninfo)
 			if err != nil {
 				return nil, err
 			}
 			return c, nil
+		},
+	}
 }
 
 func init() {