Dev-iL commented on code in PR #57532:
URL: https://github.com/apache/airflow/pull/57532#discussion_r2484350327
##########
airflow-core/src/airflow/models/xcom.py:
##########
@@ -397,7 +398,7 @@ class LazyXComSelectSequence(LazySelectSequence[Any]):
"""
@staticmethod
- def _rebuild_select(stmt: TextClause) -> Select[tuple[Any]]:
+ def _rebuild_select(stmt: TextClause) -> Any:
Review Comment:
Is there really nothing that can be said about the return type? Using `Any`
feels to me like a non-solution along the lines of ignore or cast. At the very
least let's use `sqlalchemy.sql.Executable` (needs to be modified in the
superclass too).
--
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]
