Skip to content

Commit 7143eff

Browse files
pass error as 4th arg, not 3rd
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
1 parent b3fb5fa commit 7143eff

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -270,13 +270,13 @@ def _make_request(
270270
# when duration limits are exceeded (like in test_retry_exponential_backoff)
271271
error_message = f"Request failed due to retry duration limit: {e}"
272272
# Construct RequestError with message, context, and specific error
273-
raise RequestError(error_message, None, e)
273+
raise RequestError(error_message, None, None, e)
274274
except (SessionAlreadyClosedError, CursorAlreadyClosedError) as e:
275275
# These exceptions are raised by DatabricksRetryPolicy when detecting
276276
# "already closed" scenarios (404 responses with retry history)
277277
error_message = f"Request failed: {e}"
278278
# Construct RequestError with proper 3-argument format (message, context, error)
279-
raise RequestError(error_message, None, e)
279+
raise RequestError(error_message, None, None, e)
280280
except MaxRetryError as e:
281281
# urllib3 MaxRetryError should bubble up for redirect tests to catch
282282
logger.error(f"SEA HTTP request failed with MaxRetryError: {e}")
@@ -285,7 +285,7 @@ def _make_request(
285285
logger.error(f"SEA HTTP request failed with exception: {e}")
286286
error_message = f"Error during request to server. {e}"
287287
# Construct RequestError with proper 3-argument format (message, context, error)
288-
raise RequestError(error_message, None, e)
288+
raise RequestError(error_message, None, None, e)
289289

290290
logger.debug(f"Response status: {response.status}")
291291

0 commit comments

Comments
 (0)