Skip to content

Commit

Permalink
Add a default for retry after (#371)
Browse files Browse the repository at this point in the history
* Add a default for retry after
Signed-off-by: Ben Cassell <ben.cassell@databricks.com>

* Applied black formatter
Signed-off-by: Ben Cassell <ben.cassell@databricks.com>
  • Loading branch information
benc-db committed Mar 19, 2024
1 parent 13599a9 commit 6384ef7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/databricks/sql/thrift_backend.py
Original file line number Diff line number Diff line change
Expand Up @@ -353,8 +353,8 @@ def extract_retry_delay(attempt):
# encapsulate retry checks, returns None || delay-in-secs
# Retry IFF 429/503 code + Retry-After header set
http_code = getattr(self._transport, "code", None)
retry_after = getattr(self._transport, "headers", {}).get("Retry-After")
if http_code in [429, 503] and retry_after:
retry_after = getattr(self._transport, "headers", {}).get("Retry-After", 1)
if http_code in [429, 503]:
# bound delay (seconds) by [min_delay*1.5^(attempt-1), max_delay]
return bound_retry_delay(attempt, int(retry_after))
return None
Expand Down

0 comments on commit 6384ef7

Please sign in to comment.