lazerg opened a new pull request, #76335:
URL: https://github.com/apache/airflow/pull/76335
<!-- SPDX-License-Identifier: Apache-2.0
https://www.apache.org/licenses/LICENSE-2.0 -->
The Amazon DuckDB hook ran `import duckdb` before importing `DuckDBHook`, so
loading it without the DuckDB provider still imported `duckdb` before the hook
was skipped. The 2.11 webserver loads every connection-type hook in the
gunicorn master before forking, and `duckdb` 1.4/1.5 starts a thread pool on
import, so the forked workers segfault on exit.
The module now imports only `DuckDBHook` at the top, and `configure_secrets`
imports `duckdb` where it needs `duckdb.Error`. The new test blocks the DuckDB
provider and checks that `duckdb` is not imported. It fails without this change.
closes: #76331
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes (Claude Code, Claude Opus 5.5)
Generated-by: Claude Code (Claude Opus 5.5) following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
--
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]
