fixing bug #138 (#140)

next integration test will include checks against future regression.
twitter · Feb 26, 2017 · 1738c2f · 1738c2f
1 parent 23ba44e
commit 1738c2f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/protocol/data/memcache/request.c b/src/protocol/data/memcache/request.c
@@ -68,6 +68,7 @@ request_create(void)
     request_reset(req);
 
     INCR(request_metrics, request_create);
+    INCR(request_metrics, request_free);
 
     return req;
 }
@@ -120,7 +121,6 @@ request_pool_create(uint32_t max)
         log_crit("cannot preallocate request pool, OOM. abort");
         exit(EXIT_FAILURE);
     }
-    UPDATE_VAL(request_metrics, request_free, max);
 }
 
 struct request *

diff --git a/src/protocol/data/memcache/response.c b/src/protocol/data/memcache/response.c
@@ -55,6 +55,7 @@ response_create(void)
     response_reset(rsp);
 
     INCR(response_metrics, response_create);
+    INCR(response_metrics, response_free);
 
     return rsp;
 }
@@ -106,7 +107,6 @@ response_pool_create(uint32_t max)
         log_crit("cannot preallocate response pool, OOM. abort");
         exit(EXIT_FAILURE);
     }
-    UPDATE_VAL(response_metrics, response_free, max);
 }
 
 struct response *