From a8dd211acff8f996d5afa2e39a2bb147da204231 Mon Sep 17 00:00:00 2001 From: Mario de Frutos Date: Mon, 4 Apr 2016 10:11:37 +0200 Subject: [PATCH] Use retry on timeout flag for Redis connections In order to avoid some fails due some high load peaks in our Redis databases, we are going to active this flag that retries 1 more time in case it receives a socket.Timeout error while reading or connecting --- .../cartodb_services/tools/redis_tools.py | 11 +++++------ server/lib/python/cartodb_services/setup.py | 2 +- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/server/lib/python/cartodb_services/cartodb_services/tools/redis_tools.py b/server/lib/python/cartodb_services/cartodb_services/tools/redis_tools.py index 24ba3c8..2b77531 100644 --- a/server/lib/python/cartodb_services/cartodb_services/tools/redis_tools.py +++ b/server/lib/python/cartodb_services/cartodb_services/tools/redis_tools.py @@ -16,14 +16,13 @@ class RedisConnection: sentinel = Sentinel([(self._config.host, self._config.port)], socket_timeout=self._config.timeout) - return sentinel.master_for( - self._config.sentinel_id, - socket_timeout=self._config.timeout, - db=self._config.db - ) + return sentinel.master_for(self._config.sentinel_id, + socket_timeout=self._config.timeout, + db=self._config.db, + retry_on_timeout=True) else: conn = StrictRedis(host=self._config.host, port=self._config.port, - db=self._config.db, + db=self._config.db, retry_on_timeout=True, socket_timeout=self._config.timeout) return conn diff --git a/server/lib/python/cartodb_services/setup.py b/server/lib/python/cartodb_services/setup.py index 9285caa..7c66b55 100644 --- a/server/lib/python/cartodb_services/setup.py +++ b/server/lib/python/cartodb_services/setup.py @@ -10,7 +10,7 @@ from setuptools import setup, find_packages setup( name='cartodb_services', - version='0.4.2', + version='0.4.3', description='CartoDB Services API Python Library',