Jkhall81 commented on code in PR #59915:
URL: https://github.com/apache/airflow/pull/59915#discussion_r2652945434
##########
providers/exasol/src/airflow/providers/exasol/hooks/exasol.py:
##########
@@ -96,6 +100,10 @@ def sqlalchemy_url(self) -> URL:
:return: the extracted sqlalchemy.engine.URL object.
"""
+ if URL is None:
+ raise AirflowOptionalProviderFeatureException(
Review Comment:
@jscheffl I added a [project.optional-dependencies] section to the
pyproject.toml. I greped around and found an example where this is used, so I
think I got the format right.
@sunank200 Regarding the error message: I went ahead and left the error
message as is for now. Since in this file sqlalchemy is being used to generate
a DB connection URI. Mentioning 'SQL filter clauses' might be a little
confusing.
--
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]
