Drop tenacity dependency; use google.api.core.retry in google.api.core.future (googleapis#3837)

Author: Jon Wayne Parrott

core/google/api/core/future/polling.py

@@ -16,16 +16,18 @@
 
 import abc
 import concurrent.futures
-import functools
-import operator
-
-import six
-import tenacity
 
+from google.api.core import exceptions
+from google.api.core import retry
 from google.api.core.future import _helpers
 from google.api.core.future import base
 
 
+class _OperationNotComplete(Exception):
+    """Private exception used for polling via retry."""
+    pass
+
+
 class PollingFuture(base.Future):
     """A Future that needs to poll some service to check its status.
 
@@ -55,6 +57,11 @@ def done(self):
         # pylint: disable=redundant-returns-doc, missing-raises-doc
         raise NotImplementedError()
 
+    def _done_or_raise(self):
+        """Check if the future is done and raise if it's not."""
+        if not self.done():
+            raise _OperationNotComplete()
+
     def running(self):
         """True if the operation is currently running."""
         return not self.done()
@@ -69,29 +76,16 @@ def _blocking_poll(self, timeout=None):
         if self._result_set:
             return
 
-        retry_on = tenacity.retry_if_result(
-            functools.partial(operator.is_not, True))
-        # Use exponential backoff with jitter.
-        wait_on = (
-            tenacity.wait_exponential(multiplier=1, max=10) +
-            tenacity.wait_random(0, 1))
-
-        if timeout is None:
-            retry = tenacity.retry(retry=retry_on, wait=wait_on)
-        else:
-            retry = tenacity.retry(
-                retry=retry_on,
-                wait=wait_on,
-                stop=tenacity.stop_after_delay(timeout))
+        retry_ = retry.Retry(
+            predicate=retry.if_exception_type(_OperationNotComplete),
+            deadline=timeout)
 
         try:
-            retry(self.done)()
-        except tenacity.RetryError as exc:
-            six.raise_from(
-                concurrent.futures.TimeoutError(
-                    'Operation did not complete within the designated '
-                    'timeout.'),
-                exc)
+            retry_(self._done_or_raise)()
+        except exceptions.RetryError:
+            raise concurrent.futures.TimeoutError(
+                'Operation did not complete within the designated '
+                'timeout.')
 
     def result(self, timeout=None):
         """Get the result of the operation, blocking if necessary.

core/setup.py

@@ -57,7 +57,6 @@
     'requests >= 2.18.0, < 3.0.0dev',
     'setuptools >= 34.0.0',
     'six',
-    'tenacity >= 4.0.0, <5.0.0dev'
 ]
 
 EXTRAS_REQUIREMENTS = {