Remove SQLA 1 limit in Fab provider

[vincbeck]

See what breaks.

---

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

[potiuk]

Getting closer?

[vincbeck]

Getting closer?

  Found 525 errors in 100 files (checked 996 source files)

Kinda :) I'll create an issue to get help from the community to resolve all these static errors

[vincbeck]

@potiuk should we set sqlalchemy>=2? Min SQLAlchemy tests are failing because there is a conflict with redshift-sqlalchemy, which is surprising because we do not have this issue today.

[potiuk]

@potiuk should we set sqlalchemy>=2? Min SQLAlchemy tests are failing because there is a conflict with redshift-sqlalchemy, which is surprising because we do not have this issue today.

Yes. I think that would be a good idea. There is nothing holding us with SQLalchemy 1 for Airflow 3.2

[vincbeck]

All tests are passing but mypy-airflow-core and mypy-task-sdk 🥳

[vincbeck]

Closing in favor of #58049

[Prab-27]

Sorry, I accidentally comment on this file while referencing it.