Avoid deepcopy of immutables to improve performance by 2-6x

redis · May 9, 2024 · ff1a971 · ff1a971
1 parent 2c94569
commit ff1a971
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/redis/_cache.py b/redis/_cache.py
@@ -159,6 +159,8 @@ class EvictionPolicy(Enum):
 _CTIME = "ctime"
 _ACCESS_COUNT = "access_count"
 
+IMMUTABLES = (bytes, str, int, float, bool, type(None), tuple)
+
 
 class AbstractCache(ABC):
     """
@@ -267,7 +269,12 @@ def get(self, command: Union[str, Sequence[str]]) -> ResponseT:
                 self.delete_command(command)
                 return
             self._update_access(command)
-            return copy.deepcopy(self.cache[command]["response"])
+            response = self.cache[command]["response"]
+            return (
+                response
+                if isinstance(response, IMMUTABLES)
+                else copy.deepcopy(response)
+            )
 
     def delete_command(self, command: Union[str, Sequence[str]]):
         """