stop using redis single connection as it is not thread safe

Intsights · Oct 3, 2021 · 8368ee5 · 8368ee5
1 parent 9ab1f67
commit 8368ee5
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -1,14 +1,14 @@
 exclude: '.*(test|benchmark).*'
 repos:
 -   repo: https://github.com/Intsights/flake8-intsights
-    rev: v0.1.5
+    rev: v0.1.6
     hooks:
     -   id: flake8-intsights
 -   repo: https://github.com/PyCQA/isort
-    rev: 5.7.0
+    rev: 5.9.3
     hooks:
     -   id: isort
 -   repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v0.812
+    rev: v0.910
     hooks:
     -   id: mypy
diff --git a/requirements.txt b/requirements.txt
@@ -1,4 +1,4 @@
-hiredis==1.*
+hiredis==2.*
 msgpack==1.*
 orjson==3.*
 psutil==5.*

diff --git a/sergeant/connector/redis.py b/sergeant/connector/redis.py
@@ -228,7 +228,7 @@ def __init__(
                 socket_keepalive=True,
                 socket_connect_timeout=10,
                 socket_timeout=60,
-                single_connection_client=True,
+                single_connection_client=False,
             )
             for node in nodes
         ]

diff --git a/setup.py b/setup.py
@@ -3,7 +3,7 @@
 
 setuptools.setup(
     name='sergeant',
-    version='0.21.1',
+    version='0.22.0',
     author='Gal Ben David',
     author_email='[email protected]',
     url='https://github.com/Intsights/sergeant',
@@ -24,7 +24,7 @@
     python_requires='>=3.7',
     zip_safe=False,
     install_requires=[
-        'hiredis==1.*',
+        'hiredis==2.*',
         'msgpack==1.*',
         'orjson==3.*',
         'psutil==5.*',