Changed defaults to correct values

Author: vladvildanov

redis/client.py

@@ -216,8 +216,8 @@ def __init__(
         use_cache: bool = False,
         cache: Optional[CacheInterface] = None,
         cache_eviction: Optional[EvictionPolicy] = None,
-        cache_size: int = 128,
-        cache_ttl: int = 300,
+        cache_size: int = 10000,
+        cache_ttl: int = 0,
     ) -> None:
         """
         Initialize a new Redis client.

redis/cluster.py

@@ -508,8 +508,8 @@ def __init__(
         use_cache: bool = False,
         cache: Optional[CacheInterface] = None,
         cache_eviction: Optional[EvictionPolicy] = None,
-        cache_size: int = 128,
-        cache_ttl: int = 300,
+        cache_size: int = 10000,
+        cache_ttl: int = 0,
         **kwargs,
     ):
         """
@@ -1336,8 +1336,8 @@ def __init__(
         use_cache: bool = False,
         cache: Optional[CacheInterface] = None,
         cache_eviction: Optional[EvictionPolicy] = None,
-        cache_size: int = 128,
-        cache_ttl: int = 300,
+        cache_size: int = 10000,
+        cache_ttl: int = 0,
         **kwargs,
     ):
         self.nodes_cache = {}

redis/connection.py

@@ -734,10 +734,7 @@ class CacheProxyConnection(ConnectionInterface):
     KEYS_MAPPING_CACHE_SIZE = 10000
 
     def __init__(
-        self,
-        conn: ConnectionInterface,
-        cache: CacheInterface,
-        conf: CacheConfiguration
+        self, conn: ConnectionInterface, cache: CacheInterface, conf: CacheConfiguration
     ):
         self.pid = os.getpid()
         self._conn = conn
@@ -830,7 +827,8 @@ def read_response(
             # Check if command response exists in a cache and it's not in progress.
             if (
                 self._cache.exists(self._current_command_hash)
-                and self._cache.get(self._current_command_hash) != self.CACHE_DUMMY_STATUS
+                and self._cache.get(self._current_command_hash)
+                != self.CACHE_DUMMY_STATUS
             ):
                 return copy.deepcopy(self._cache.get(self._current_command_hash))
 
@@ -1444,7 +1442,7 @@ def make_connection(self) -> "ConnectionInterface":
             return CacheProxyConnection(
                 self.connection_class(**self.connection_kwargs),
                 self.cache,
-                self._cache_conf
+                self._cache_conf,
             )
 
         return self.connection_class(**self.connection_kwargs)

tests/test_cache.py

@@ -38,7 +38,7 @@ def r(request):
 
 @pytest.mark.skipif(HIREDIS_AVAILABLE, reason="PythonParser only")
 @pytest.mark.onlynoncluster
-@skip_if_resp_version(2)
+# @skip_if_resp_version(2)
 class TestCache:
     @pytest.mark.parametrize(
         "r",