update retry classes: remove slots & move __eq__ to concrete classes

Author: terencehonles

redis/asyncio/retry.py

@@ -13,6 +13,16 @@ class Retry(AbstractRetry):
         TimeoutError,
     )
 
+    def __eq__(self, other: Any) -> bool:
+        if not isinstance(other, Retry):
+            return NotImplemented
+
+        return (
+            self._backoff == other._backoff
+            and self._retries == other._retries
+            and set(self._supported_errors) == set(other._supported_errors)
+        )
+
     async def call_with_retry(
         self, do: Callable[[], Awaitable[T]], fail: Callable[[Exception], Any]
     ) -> T:

redis/retry.py

@@ -13,7 +13,6 @@
 class AbstractRetry:
     """Retry a specific number of times after a failure"""
 
-    __slots__ = "_backoff", "_retries", "_supported_errors"
     _supported_errors: Tuple[Type[Exception], ...]
 
     def __init__(
@@ -34,16 +33,6 @@ def __init__(
         if supported_errors:
             self._supported_errors = supported_errors
 
-    def __eq__(self, other: Any) -> bool:
-        if not isinstance(other, AbstractRetry):
-            return NotImplemented
-
-        return (
-            self._backoff == other._backoff
-            and self._retries == other._retries
-            and set(self._supported_errors) == set(other._supported_errors)
-        )
-
     def __hash__(self) -> int:
         return hash((self._backoff, self._retries, frozenset(self._supported_errors)))
 
@@ -77,6 +66,16 @@ class Retry(AbstractRetry):
         socket.timeout,
     )
 
+    def __eq__(self, other: Any) -> bool:
+        if not isinstance(other, Retry):
+            return NotImplemented
+
+        return (
+            self._backoff == other._backoff
+            and self._retries == other._retries
+            and set(self._supported_errors) == set(other._supported_errors)
+        )
+
     def call_with_retry(
         self,
         do: Callable[[], T],