potiuk commented on code in PR #60062:
URL: https://github.com/apache/airflow/pull/60062#discussion_r2659686489
##########
providers/apache/impala/src/airflow/providers/apache/impala/hooks/impala.py:
##########
@@ -19,14 +19,26 @@
from typing import TYPE_CHECKING
from impala.dbapi import connect
-from sqlalchemy.engine import URL
+from airflow.exceptions import AirflowOptionalProviderFeatureException
from airflow.providers.common.sql.hooks.sql import DbApiHook
if TYPE_CHECKING:
from impala.interface import Connection
+def _get_sqlalchemy_url_class():
Review Comment:
Type hinting is needed here as well (See the other PRs)
--
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]
