bug 1089782 - fix bad param usage

mozilla · Oct 28, 2014 · 0d598d2 · 0d598d2
1 parent c2bc149
commit 0d598d2
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/treeherder/model/utils.py b/treeherder/model/utils.py
@@ -59,12 +59,11 @@ def where_wolf(project, flat_exclusions):
     return " AND ({0})".format(condition_list), values_list
 
 
-def retry_execute(dhub, logger, **kwargs):
+def retry_execute(dhub, logger, retries=0, **kwargs):
     """Retry the query in the case of an OperationalError."""
     try:
         return dhub.execute(**kwargs)
     except OperationalError:
-        retries = kwargs.get('retries', 0) + 1
 
         if retries < 20:
             sleep_time = round(random.random() * .05, 3)  # 0 to 50ms
@@ -74,7 +73,6 @@ def retry_execute(dhub, logger, **kwargs):
                         retries, sleep_time, kwargs
                     ))
             time.sleep(sleep_time)
-            kwargs['retries'] = retries
-            return retry_execute(dhub, logger, **kwargs)
+            return retry_execute(dhub, logger, retries, **kwargs)
         else:
             raise