Dev-iL commented on issue #28723: URL: https://github.com/apache/airflow/issues/28723#issuecomment-3004935545
If anyone wants to help move this issue along, you might want to base your work off of my branch mentioned in the PR right above this comment - which adds a CI job that runs all tests with SQLA v2. Locally, these tests can be run using ```bash breeze testing core-tests --upgrade-sqlalchemy --maxfail=100 # Expect many failures initially ``` Currently, most errors are of the below sort: > ERROR airflow-core/tests/unit/always/test_secrets.py - sqlalchemy.orm.exc.MappedAnnotationError: Type annotation for "TaskInstance.dag_model" can't be correctly interpreted for Annotated Declarative Table form. ORM annotations should normally make use of the ``Mapped[]`` generic type, or other ORM-compatible generic type, as a container for the actual type, which indicates the intent that the attribute is mapped. Class variables that are not intended to be mapped by the ORM should use ClassVar[]. To allow Annotated Declarative to disregard legacy annotations which don't use Mapped[] to pass, set "__allow_unmapped__ = True" on the class or a superclass this class. (Background on this error at: https://sqlalche.me/e/20/zlpr) Here's an overview of this problem and potential solutions by Claude: [sqlalchemy-upgrade-guide.md](https://github.com/user-attachments/files/20905444/sqlalchemy-upgrade-guide.md) -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
