You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openwhisk.apache.org by mh...@apache.org on 2018/08/04 21:42:31 UTC

[incubator-openwhisk-apigateway] 01/01: Perf: Remove keepalive thresholds

This is an automated email from the ASF dual-hosted git repository.

mhamann pushed a commit to branch redis-perf
in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-apigateway.git

commit 25ceb61886f12eb609255ceec034988e7cf179e2
Author: Matt Hamann <mh...@us.ibm.com>
AuthorDate: Sat Aug 4 17:42:22 2018 -0400

    Perf: Remove keepalive thresholds
---
 scripts/lua/lib/redis.lua | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/scripts/lua/lib/redis.lua b/scripts/lua/lib/redis.lua
index b847917..1828b83 100644
--- a/scripts/lua/lib/redis.lua
+++ b/scripts/lua/lib/redis.lua
@@ -22,6 +22,7 @@ local cjson = require "cjson"
 local utils = require "lib/utils"
 local logger = require "lib/logger"
 local request = require "lib/request"
+local redis = require "resty.redis"
 local lrucache
 local CACHE_SIZE
 local CACHE_TTL
@@ -62,7 +63,6 @@ function _M.init()
   local host = REDIS_HOST
   local password = REDIS_PASS
   local port = REDIS_PORT
-  local redis = require "resty.redis"
   local red = redis:new()
   red:set_timeout(REDIS_TIMEOUT)
   -- Connect to Redis server
@@ -94,8 +94,8 @@ end
 --- Add current redis connection in the ngx_lua cosocket connection pool
 -- @param red Redis client instance
 function _M.close(red)
-  -- put it into the connection pool of size 100, with 10 seconds max idle time
-  local ok, err = red:set_keepalive(10000, 100)
+  -- release connection into the pool
+  local ok, err = red:set_keepalive()
   if not ok then
     request.err(500, utils.concatStrings({"Failed to set keepalive: ", err}))
   end