This is an automated email from the ASF dual-hosted git repository.
gopidesu pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 64e7408cd35 Fix unreachable code mypy warnings in teradata provider
(#53488)
64e7408cd35 is described below
commit 64e7408cd35cce3ae8a90b2cf17b154e0d1fde67
Author: fweilun <[email protected]>
AuthorDate: Sat Jul 26 02:29:59 2025 +0800
Fix unreachable code mypy warnings in teradata provider (#53488)
* include list[str] in DbApiHook.run return type to resolve MyPy
unreachable warnings
* fix: update dialect.py return type to match run()
* generalize SQL result type from list[str] to list for broader
compatibility
---
.../common/sql/src/airflow/providers/common/sql/dialects/dialect.py | 2 +-
.../common/sql/src/airflow/providers/common/sql/dialects/dialect.pyi | 2 +-
providers/common/sql/src/airflow/providers/common/sql/hooks/sql.py | 4 ++--
providers/common/sql/src/airflow/providers/common/sql/hooks/sql.pyi | 2 +-
4 files changed, 5 insertions(+), 5 deletions(-)
diff --git
a/providers/common/sql/src/airflow/providers/common/sql/dialects/dialect.py
b/providers/common/sql/src/airflow/providers/common/sql/dialects/dialect.py
index 6380f4091fd..44dd397817b 100644
--- a/providers/common/sql/src/airflow/providers/common/sql/dialects/dialect.py
+++ b/providers/common/sql/src/airflow/providers/common/sql/dialects/dialect.py
@@ -158,7 +158,7 @@ class Dialect(LoggingMixin):
handler: Callable[[Any], T] | None = None,
split_statements: bool = False,
return_last: bool = True,
- ) -> tuple | list[tuple] | list[list[tuple] | tuple] | None:
+ ) -> tuple | list | list[tuple] | list[list[tuple] | tuple] | None:
return self.hook.run(sql, autocommit, parameters, handler,
split_statements, return_last)
def get_records(
diff --git
a/providers/common/sql/src/airflow/providers/common/sql/dialects/dialect.pyi
b/providers/common/sql/src/airflow/providers/common/sql/dialects/dialect.pyi
index 7ab9a3efa0f..0945ecb43d0 100644
--- a/providers/common/sql/src/airflow/providers/common/sql/dialects/dialect.pyi
+++ b/providers/common/sql/src/airflow/providers/common/sql/dialects/dialect.pyi
@@ -72,7 +72,7 @@ class Dialect(LoggingMixin):
handler: Callable[[Any], T] | None = None,
split_statements: bool = False,
return_last: bool = True,
- ) -> tuple | list[tuple] | list[list[tuple] | tuple] | None: ...
+ ) -> tuple | list | list[tuple] | list[list[tuple] | tuple] | None: ...
def get_records(
self, sql: str | list[str], parameters: Iterable | Mapping[str, Any] |
None = None
) -> Any: ...
diff --git a/providers/common/sql/src/airflow/providers/common/sql/hooks/sql.py
b/providers/common/sql/src/airflow/providers/common/sql/hooks/sql.py
index 86c01d396da..7dbda88aa93 100644
--- a/providers/common/sql/src/airflow/providers/common/sql/hooks/sql.py
+++ b/providers/common/sql/src/airflow/providers/common/sql/hooks/sql.py
@@ -680,7 +680,7 @@ class DbApiHook(BaseHook):
handler: Callable[[Any], T] = ...,
split_statements: bool = ...,
return_last: bool = ...,
- ) -> tuple | list[tuple] | list[list[tuple] | tuple] | None: ...
+ ) -> tuple | list | list[tuple] | list[list[tuple] | tuple] | None: ...
def run(
self,
@@ -690,7 +690,7 @@ class DbApiHook(BaseHook):
handler: Callable[[Any], T] | None = None,
split_statements: bool = False,
return_last: bool = True,
- ) -> tuple | list[tuple] | list[list[tuple] | tuple] | None:
+ ) -> tuple | list | list[tuple] | list[list[tuple] | tuple] | None:
"""
Run a command or a list of commands.
diff --git
a/providers/common/sql/src/airflow/providers/common/sql/hooks/sql.pyi
b/providers/common/sql/src/airflow/providers/common/sql/hooks/sql.pyi
index 9c792e1bd82..81ff5f5b8e7 100644
--- a/providers/common/sql/src/airflow/providers/common/sql/hooks/sql.pyi
+++ b/providers/common/sql/src/airflow/providers/common/sql/hooks/sql.pyi
@@ -204,4 +204,4 @@ class DbApiHook(BaseHook):
handler: Callable[[Any], T] = ...,
split_statements: bool = ...,
return_last: bool = ...,
- ) -> tuple | list[tuple] | list[list[tuple] | tuple] | None: ...
+ ) -> tuple | list | list[tuple] | list[list[tuple] | tuple] | None: ...
