potiuk opened a new pull request, #63395: URL: https://github.com/apache/airflow/pull/63395
The `object_storage_provider.py` in the `common.sql` provider has top-level imports from `datafusion.object_store` which is not a required dependency. This breaks the `Validate provider.yaml files` CI check because the module fails to import when `datafusion` is not installed. Move the `from datafusion.object_store import AmazonS3` and `from datafusion.object_store import LocalFileSystem` imports inside their respective `create_object_store()` methods where they are actually used, following the lazy-import pattern already used elsewhere in the codebase. --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Claude Opus 4.6) Generated-by: Claude Code (Claude Opus 4.6) 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]
