Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Updated retry timeout#497

Merged
jprakash-db merged 2 commits into
release/3.7.2databricks/databricks-sql-python:release/3.7.2from
jprakash-db/updated-retrydatabricks/databricks-sql-python:jprakash-db/updated-retryCopy head branch name to clipboard
Jan 31, 2025
Merged

Updated retry timeout#497
jprakash-db merged 2 commits into
release/3.7.2databricks/databricks-sql-python:release/3.7.2from
jprakash-db/updated-retrydatabricks/databricks-sql-python:jprakash-db/updated-retryCopy head branch name to clipboard

Conversation

@jprakash-db

Copy link
Copy Markdown
Contributor

Description

  • Set the retry timeout to be the max of retry_after and retry_max_delay
  • Increased the retry_max_delay

@benc-db

benc-db commented Jan 31, 2025

Copy link
Copy Markdown
Collaborator

Going to test with my local repro to verify the fix.

@benc-db

benc-db commented Jan 31, 2025

Copy link
Copy Markdown
Collaborator

Looks reasonable when I test with dbt-databricks. Can we add a debug log statement that says how long we're going to sleep for?

@jprakash-db

Copy link
Copy Markdown
Contributor Author

@benc-db Added the logging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.