Skip to content

Commit 112958d

Browse files
simplify error handling (attempt)
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
1 parent e3528a1 commit 112958d

File tree

1 file changed

+0
-12
lines changed

1 file changed

+0
-12
lines changed

src/databricks/sql/backend/sea/utils/http_client.py

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -269,18 +269,6 @@ def _make_request(
269269
preload_content=False,
270270
retries=self.retry_policy,
271271
)
272-
except MaxRetryDurationError as e:
273-
# MaxRetryDurationError is raised directly by DatabricksRetryPolicy
274-
# when duration limits are exceeded (like in test_retry_exponential_backoff)
275-
error_message = f"Request failed due to retry duration limit: {e}"
276-
# Construct RequestError with message, context, and specific error
277-
raise RequestError(error_message, None, None, e)
278-
except (SessionAlreadyClosedError, CursorAlreadyClosedError) as e:
279-
# These exceptions are raised by DatabricksRetryPolicy when detecting
280-
# "already closed" scenarios (404 responses with retry history)
281-
error_message = f"Request failed: {e}"
282-
# Construct RequestError with proper 3-argument format (message, context, error)
283-
raise RequestError(error_message, None, None, e)
284272
except MaxRetryError as e:
285273
# urllib3 MaxRetryError should bubble up for redirect tests to catch
286274
logger.error(f"SEA HTTP request failed with MaxRetryError: {e}")

0 commit comments

Comments
 (0)