AC review

Author: NoahStapp

pymongo/asynchronous/mongo_client.py

@@ -2853,9 +2853,8 @@ async def run(self) -> T:
                         exc_code = getattr(exc, "code", None)
                         overloaded = exc.has_error_label("SystemOverloadedError")
                         always_retryable = exc.has_error_label("RetryableError") and overloaded
-                        if (
-                            not self._client.options.retry_reads
-                            or not always_retryable
+                        if not self._client.options.retry_reads or (
+                            not always_retryable
                             and (
                                 self._is_not_eligible_for_retry()
                                 or (
@@ -2934,7 +2933,7 @@ async def run(self) -> T:
 
                 self._always_retryable = always_retryable
                 if overloaded:
-                    delay = self._retry_policy.backoff(self._attempt_number) if overloaded else 0
+                    delay = self._retry_policy.backoff(self._attempt_number)
                     if not await self._retry_policy.should_retry(self._attempt_number, delay):
                         if exc_to_check.has_error_label("NoWritesPerformed") and self._last_error:
                             raise self._last_error from exc

pymongo/synchronous/mongo_client.py

@@ -2843,9 +2843,8 @@ def run(self) -> T:
                         exc_code = getattr(exc, "code", None)
                         overloaded = exc.has_error_label("SystemOverloadedError")
                         always_retryable = exc.has_error_label("RetryableError") and overloaded
-                        if (
-                            not self._client.options.retry_reads
-                            or not always_retryable
+                        if not self._client.options.retry_reads or (
+                            not always_retryable
                             and (
                                 self._is_not_eligible_for_retry()
                                 or (
@@ -2924,7 +2923,7 @@ def run(self) -> T:
 
                 self._always_retryable = always_retryable
                 if overloaded:
-                    delay = self._retry_policy.backoff(self._attempt_number) if overloaded else 0
+                    delay = self._retry_policy.backoff(self._attempt_number)
                     if not self._retry_policy.should_retry(self._attempt_number, delay):
                         if exc_to_check.has_error_label("NoWritesPerformed") and self._last_error:
                             raise self._last_error from exc