You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
ERROR:databricks.sql.common.unified_http_client:HTTP request error: Retry request would exceed Retry policy max retry duration of 300 seconds #709
I did a bunch of digging on this one as it hit our team as well.
The enabling of the circuitbreaker in 4.2.2 has caused this behavior. Previously the library allowed https:// in the path. The changes introduced in the telemetry client circuit breaker prepends https:// to the URL which causes this to break when checking the server side feature flag.
This was not an issue prior to 4.2.2 as the breaker was disabled by default but enabled in 4.2.2 with no exposure/access to the flag to turn it off by an end user.
It seems the call to check the server side feature flag is made regardless if telemetry is enabled or not, and since the flag is not is exposed I was unable to find a way to disable the feature flag check.
As a note we use this with DBT which also complicates some things.
The fix for now is to remove the https:// but this was a major change enabling telemetry that came in on a minor release. Which is kind of disheartening and not very transparent. Would love to see some clearer communication and flagging around this instead of just failing when the external feature flags fail.
Hi @tkeller-moxe
Sorry for the trouble this caused!
We updated the docs when 4.2.2 was released to cover telemetry settings. The connector has an enable_telemetry flag, but you're right that it's not accessible through dbt-databricks.
We're working on a fix to handle hostnames with https:// prefixes properly (which is common in dbt setups). We'll keep you posted once it's ready.
Ah I missed the link to the databricks documentation thanks for sharing.
Appreciate the action! Caught our team off guard and we have implemented out own workaround for now but will be nice to remove that complexity later!
Activity
tkeller-moxe commentedon Dec 29, 2025
I did a bunch of digging on this one as it hit our team as well.
The enabling of the circuitbreaker in 4.2.2 has caused this behavior. Previously the library allowed
https://in the path. The changes introduced in the telemetry client circuit breaker prepends https:// to the URL which causes this to break when checking the server side feature flag.This was not an issue prior to 4.2.2 as the breaker was disabled by default but enabled in 4.2.2 with no exposure/access to the flag to turn it off by an end user.
It seems the call to check the server side feature flag is made regardless if telemetry is enabled or not, and since the flag is not is exposed I was unable to find a way to disable the feature flag check.
As a note we use this with DBT which also complicates some things.
The fix for now is to remove the https:// but this was a major change enabling telemetry that came in on a minor release. Which is kind of disheartening and not very transparent. Would love to see some clearer communication and flagging around this instead of just failing when the external feature flags fail.
samikshya-db commentedon Dec 29, 2025
@nikhilsuri-db - please take a look on this one.
nikhilsuri-db commentedon Dec 30, 2025
Hi @tkeller-moxe
Sorry for the trouble this caused!
We updated the docs when 4.2.2 was released to cover telemetry settings. The connector has an enable_telemetry flag, but you're right that it's not accessible through dbt-databricks.
We're working on a fix to handle hostnames with https:// prefixes properly (which is common in dbt setups). We'll keep you posted once it's ready.
tkeller-moxe commentedon Dec 31, 2025
Ah I missed the link to the databricks documentation thanks for sharing.
Appreciate the action! Caught our team off guard and we have implemented out own workaround for now but will be nice to remove that complexity later!
peco-engineer-bot commentedon Jul 23, 2026
🤖 engineer-bot opened a fix PR: #878
Review before merge.