cjsekl commented on a change in pull request #9936:
URL: https://github.com/apache/airflow/pull/9936#discussion_r458967888
##########
File path: airflow/providers/vertica/hooks/vertica.py
##########
@@ -31,11 +31,11 @@ class VerticaHook(DbApiHook):
default_conn_name = 'vertica_default'
supports_autocommit = True
- def get_conn(self):
+ def get_conn(self) -> connect:
"""
- Returns verticaql connection object
+ Return verticaql connection object
"""
- conn = self.get_connection(self.vertica_conn_id) # pylint:
disable=no-member
+ conn = self.get_connection(self.vertica_conn_id) # type: ignore #
pylint: disable=no-member
Review comment:
This is ignored by pylint and mypy because `VerticaHook.vertica_conn_id`
overrides `DbApiHook.conn_name_attr`.
Is there a better way to handle this than ignoring?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
