From 2ed5a4ae86fdb9e1970e081c14f463b7a72b166a Mon Sep 17 00:00:00 2001 From: Samuel Merritt Date: Tue, 4 Mar 2014 18:40:12 -0800 Subject: [PATCH] Speed up failing InternalClient requests InternalClient.make_request was sleeping after every attempt, including the last one. We don't need to sleep after the final attempt; we can just return failure. Change-Id: Ieee5f96afb5b39a659d8438a237bfd907405165a --- swift/common/internal_client.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/swift/common/internal_client.py b/swift/common/internal_client.py index b19ba33b03..480d44b55c 100644 --- a/swift/common/internal_client.py +++ b/swift/common/internal_client.py @@ -178,7 +178,9 @@ class InternalClient(object): return resp except (Exception, Timeout): exc_type, exc_value, exc_traceback = exc_info() - sleep(2 ** (attempt + 1)) + # sleep only between tries, not after each one + if attempt < self.request_tries - 1: + sleep(2 ** (attempt + 1)) if resp: raise UnexpectedResponse( _('Unexpected response: %s') % resp.status, resp)