eladkal commented on code in PR #63518:
URL: https://github.com/apache/airflow/pull/63518#discussion_r2931350586
##########
providers/databricks/pyproject.toml:
##########
@@ -64,6 +64,7 @@ dependencies = [
"requests>=2.32.0,<3",
"databricks-sql-connector>=4.0.0",
"aiohttp>=3.9.2, <4",
+ "aiofiles>=23.2.0",
Review Comment:
The `AirflowOptionalProviderFeatureException` was created specifically for
such cases. so users will be aware that they are using an optional feature of a
provider and missing something to make it work properly.
So my proposal is to move the import from top level to the function that
actually use it and add a `AirflowOptionalProviderFeatureException` to notify
users if depended provider is missing.
Let me know if that works
--
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]
