NewSmoke38 opened a new pull request, #59916: URL: https://github.com/apache/airflow/pull/59916
I modified the Drill provider to safely handle the absence of `sqlalchemy` by wrapping the import in a try-except block. It now raises `AirflowOptionalProviderFeatureException` in `get_conn` if the library is missing when actually needed. **Testing:** I verified this manually by running a script that simulates a missing `sqlalchemy` environment. This confirmed that while the parent `DbApiHook` typically relies on `sqlalchemy`, the Drill provider now loads successfully without it and only raises the correct exception when `get_conn` is called. I look forward to your feedback. closes: #59897 -- 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]
