Skip to content

Unexpected argument backoff_max with databricks-sql-connector 2.9.2 #205

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
orslow opened this issue Aug 22, 2023 · 2 comments · Fixed by #206
Closed

Unexpected argument backoff_max with databricks-sql-connector 2.9.2 #205

orslow opened this issue Aug 22, 2023 · 2 comments · Fixed by #206
Assignees

Comments

@orslow
Copy link

orslow commented Aug 22, 2023

Hello,

Release note says "databricks-sql-connector 2.9.2 is compatible with urllib3>=1.0.0", but it still pass backoff_max argument, which was introduced from urllib3 2.0.0 (ref: https://github.com/urllib3/urllib3/blob/6446fef0cf432ca035169602a1447a0d8ef53e80/src/urllib3/util/retry.py#L209)

    return sql.connect(
  File "/usr/local/lib/python3.10/site-packages/databricks/sql/__init__.py", line 50, in connect
    return Connection(server_hostname, http_path, access_token, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/databricks/sql/client.py", line 186, in __init__
    self.thrift_backend = ThriftBackend(
  File "/usr/local/lib/python3.10/site-packages/databricks/sql/thrift_backend.py", line 189, in __init__
    self.retry_policy = databricks.sql.auth.thrift_http_client.DatabricksRetryPolicy(
  File "/usr/local/lib/python3.10/site-packages/databricks/sql/auth/retry.py", line 132, in __init__
    super().__init__(
TypeError: Retry.__init__() got an unexpected keyword argument 'backoff_max'
@susodapop susodapop self-assigned this Aug 23, 2023
@susodapop
Copy link
Contributor

Thanks for this report. Fix will be released tomorrow.

@susodapop
Copy link
Contributor

Fix is incorporated into this release: https://github.com/databricks/databricks-sql-python/releases/tag/v2.9.3

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 a pull request may close this issue.

2 participants