sunank200 commented on code in PR #59916:
URL: https://github.com/apache/airflow/pull/59916#discussion_r2652349998
##########
providers/apache/drill/src/airflow/providers/apache/drill/hooks/drill.py:
##########
@@ -52,6 +55,11 @@ class DrillHook(DbApiHook):
def get_conn(self) -> PoolProxiedConnection:
"""Establish a connection to Drillbit."""
conn_md = self.get_connection(self.get_conn_id())
+
+ if create_engine is None:
+
+ raise AirflowOptionalProviderFeatureException("The 'sqlalchemy'
library is required to use this hook.")
Review Comment:
```suggestion
raise AirflowOptionalProviderFeatureException(
"sqlalchemy is required for SQL filter clause generation. "
"Install it with: pip install
'apache-airflow-providers-drill[sqlalchemy]'"
)
```
##########
providers/apache/drill/src/airflow/providers/apache/drill/hooks/drill.py:
##########
@@ -52,6 +55,11 @@ class DrillHook(DbApiHook):
def get_conn(self) -> PoolProxiedConnection:
"""Establish a connection to Drillbit."""
conn_md = self.get_connection(self.get_conn_id())
+
+ if create_engine is None:
+
Review Comment:
```suggestion
```
--
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]
