Merge pull request #202 from CartoDB/CDB-3135

Upgrades dependencies to get redis-mpool improvements

NEWS.md

@@ -1,5 +1,12 @@
 1.14.1 -- 2014-mm-dd
 --------------------
+Enhancements:
+ - Upgrades dependencies:
+    - redis-mpool
+    - cartodb-redis
+    - windshaft
+ - Specifies name in the redis pool
+ - Slow pool configuration in example configurations
 
 1.14.0 -- 2014-08-07
 --------------------

config/environments/development.js.example

@@ -103,7 +103,11 @@ var config = {
         // kept open by the server. The default is 50.
         max: 50,
         idleTimeoutMillis: 1, // idle time before dropping connection
-        reapIntervalMillis: 1 // time between cleanups
+        reapIntervalMillis: 1, // time between cleanups
+        slowPool: {
+            log: true, // whether a slow acquire must be logged or not
+            elapsedThreshold: 25 // the threshold to determine an slow acquire must be reported or not
+        }
     }
     ,sqlapi: {
         protocol: 'http',

config/environments/production.js.example

@@ -97,7 +97,11 @@ var config = {
         // kept open by the server. The default is 50.
         max: 50,
         idleTimeoutMillis: 30000, // idle time before dropping connection
-        reapIntervalMillis: 1000 // time between cleanups
+        reapIntervalMillis: 1000, // time between cleanups
+        slowPool: {
+            log: true, // whether a slow acquire must be logged or not
+            elapsedThreshold: 25 // the threshold to determine an slow acquire must be reported or not
+        }
     }
     ,sqlapi: {
         protocol: 'https',

config/environments/staging.js.example

@@ -97,7 +97,11 @@ var config = {
         // kept open by the server. The default is 50.
         max: 50,
         idleTimeoutMillis: 30000, // idle time before dropping connection
-        reapIntervalMillis: 1000 // time between cleanups
+        reapIntervalMillis: 1000, // time between cleanups
+        slowPool: {
+            log: true, // whether a slow acquire must be logged or not
+            elapsedThreshold: 25 // the threshold to determine an slow acquire must be reported or not
+        }
     }
     ,sqlapi: {
         protocol: 'https',

config/environments/test.js.example

@@ -97,7 +97,11 @@ var config = {
         // kept open by the server. The default is 50.
         max: 50,
         idleTimeoutMillis: 1, // idle time before dropping connection
-        reapIntervalMillis: 1 // time between cleanups
+        reapIntervalMillis: 1, // time between cleanups
+        slowPool: {
+            log: true, // whether a slow acquire must be logged or not
+            elapsedThreshold: 25 // the threshold to determine an slow acquire must be reported or not
+        }
     }
     ,sqlapi: {
         protocol: 'http',

lib/cartodb/cartodb_windshaft.js

@@ -2,7 +2,7 @@
 var _ = require('underscore')
     , Step       = require('step')
     , Windshaft = require('windshaft')
-    , redisPool = require('redis-mpool')(global.environment.redis)
+    , redisPool = require('redis-mpool')(_.extend(global.environment.redis, {name: 'windshaft:cartodb'}))
     // TODO: instanciate cartoData with redisPool
     , cartoData  = require('cartodb-redis')(global.environment.redis)
     , SignedMaps = require('./signed_maps.js')

npm-shrinkwrap.json

@@ -10,12 +10,12 @@
       "version": "1.3.3"
     },
     "windshaft": {
-      "version": "0.23.0",
-      "from": "https://github.com/CartoDB/Windshaft/tarball/0.23.0",
+      "version": "0.24.0",
+      "from": "https://github.com/CartoDB/Windshaft/tarball/0.24.0",
       "dependencies": {
         "grainstore": {
-          "version": "0.19.0",
-          "from": "git://github.com/CartoDB/grainstore.git#0.19.0",
+          "version": "0.20.0",
+          "from": "git://github.com/CartoDB/grainstore.git#0.20.0",
           "dependencies": {
             "carto": {
               "version": "0.9.5-cdb2",
@@ -934,12 +934,12 @@
       "version": "2.9.202"
     },
     "cartodb-redis": {
-      "version": "0.5.0",
-      "from": "git://github.com/CartoDB/node-cartodb-redis.git#0.5.0"
+      "version": "0.9.0",
+      "from": "git://github.com/CartoDB/node-cartodb-redis.git#0.9.0"
     },
     "redis-mpool": {
-      "version": "0.0.4",
-      "from": "http://github.com/CartoDB/node-redis-mpool/tarball/0.0.4",
+      "version": "0.1.0",
+      "from": "https://github.com/CartoDB/node-redis-mpool/tarball/0.1.0",
       "dependencies": {
         "generic-pool": {
           "version": "2.0.4"

package.json

@@ -24,11 +24,11 @@
     "dependencies": {
         "node-varnish": "http://github.com/Vizzuality/node-varnish/tarball/0.3.0",
         "underscore"  : "~1.3.3",
-        "windshaft": "https://github.com/CartoDB/Windshaft/tarball/0.23.0",
-        "step": "0.0.x",
+        "windshaft": "https://github.com/CartoDB/Windshaft/tarball/0.24.0",
+        "step": "~0.0.5",
         "request": "2.9.202",
-        "cartodb-redis": "git://github.com/CartoDB/node-cartodb-redis.git#0.5.0",
-        "redis-mpool": "http://github.com/CartoDB/node-redis-mpool/tarball/0.0.4",
+        "cartodb-redis": "git://github.com/CartoDB/node-cartodb-redis.git#0.9.0",
+        "redis-mpool": "https://github.com/CartoDB/node-redis-mpool/tarball/0.1.0",
         "mapnik": "http://github.com/Vizzuality/node-mapnik/tarball/0.7.26-cdb1",
         "lzma": "~1.2.3",
         "log4js": "~0.6.10",