diff --git a/.gitignore b/.gitignore
index 20276f9f..66c94734 100644
--- a/.gitignore
+++ b/.gitignore
@@ -202,3 +202,6 @@ cython_debug/
databricks_sql_connector.egg-info/
dist/
build/
+
+# vs code stuff
+.vscode
\ No newline at end of file
diff --git a/src/databricks/sql/thrift_backend.py b/src/databricks/sql/thrift_backend.py
index de505a8a..935c7711 100644
--- a/src/databricks/sql/thrift_backend.py
+++ b/src/databricks/sql/thrift_backend.py
@@ -322,27 +322,27 @@ def attempt_request(attempt):
except OSError as err:
error = err
error_message = str(err)
+ # fmt: off
+ # The built-in errno package encapsulates OSError codes, which are OS-specific.
+ # log.info for errors we believe are not unusual or unexpected. log.warn for
+ # for others like EEXIST, EBADF, ERANGE which are not expected in this context.
+ #
+ # I manually tested this retry behaviour using mitmweb and confirmed that
+ # GetOperationStatus requests are retried when I forced network connection
+ # interruptions / timeouts / reconnects. See #24 for more info.
+ # | Debian | Darwin |
+ info_errs = [ # |--------|--------|
+ errno.ESHUTDOWN, # | 32 | 32 |
+ errno.EAFNOSUPPORT, # | 97 | 47 |
+ errno.ECONNRESET, # | 104 | 54 |
+ errno.ETIMEDOUT, # | 110 | 60 |
+ ]
gos_name = TCLIServiceClient.GetOperationStatus.__name__
- if method.__name__ == gos_name:
+ # retry on timeout. Happens a lot in Azure and it is safe as data has not been sent to server yet
+ if method.__name__ == gos_name or err.errno == errno.ETIMEDOUT:
retry_delay = bound_retry_delay(attempt, self._retry_delay_default)
- # fmt: off
- # The built-in errno package encapsulates OSError codes, which are OS-specific.
- # log.info for errors we believe are not unusual or unexpected. log.warn for
- # for others like EEXIST, EBADF, ERANGE which are not expected in this context.
- #
- # I manually tested this retry behaviour using mitmweb and confirmed that
- # GetOperationStatus requests are retried when I forced network connection
- # interruptions / timeouts / reconnects. See #24 for more info.
- # | Debian | Darwin |
- info_errs = [ # |--------|--------|
- errno.ESHUTDOWN, # | 32 | 32 |
- errno.EAFNOSUPPORT, # | 97 | 47 |
- errno.ECONNRESET, # | 104 | 54 |
- errno.ETIMEDOUT, # | 110 | 60 |
- ]
-
# fmt: on
log_string = f"{gos_name} failed with code {err.errno} and will attempt to retry"
if err.errno in info_errs:
pFad - Phonifier reborn
Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.
Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.
Alternative Proxies:
Alternative Proxy
pFad Proxy
pFad v3 Proxy
pFad v4 Proxy