moomindani commented on issue #69031: URL: https://github.com/apache/airflow/issues/69031#issuecomment-4862346025
Triaged at a maintainer's request. Confirmed: `sqlalchemy_url` builds its query dict from `self._http_path` directly, which is only set from the constructor kwarg or lazily inside `get_conn()`, so an `http_path` defined in the connection extra (or via `sql_endpoint_name`) is missing from `sqlalchemy_url` / `get_uri()` until `get_conn()` has run. There is an open PR addressing this — #69037 — which takes the right approach (a shared `_resolve_http_path()` helper). I reviewed it and flagged one regression that needs resolving before merge: the `sql_endpoint_name` case must not trigger a REST API call from inside the `sqlalchemy_url` property (details in the PR review). Workaround until the fix lands: pass `http_path` to the hook explicitly, or call `get_conn()` once before `get_uri()`. --- Drafted-by: Claude Code (Fable 5); reviewed by @moomindani before posting -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
