-
Notifications
You must be signed in to change notification settings - Fork 16.6k
[v3-1-test] Protect against hanging thread in aiosqlite 0.22+ (#60217)#60245
[v3-1-test] Protect against hanging thread in aiosqlite 0.22+ (#60217)#60245potiuk merged 1 commit intoapache:v3-1-testfrom
Conversation
The change in model of aiosqlite impacted sqlalchemy handling of closing the connection - the way how currrent sqlalchemy works with it is that it leaves threads hanging. This is fixed in sqlalchemy main - sqlalchemy/sqlalchemy#13039 but until they release it and until we have the sqlalchemy>=2.0.46, we should upper-bind the aiosqlite
(cherry picked from commit 9b364ca)
^ 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.
The change in model of aiosqlite impacted sqlalchemy handling of
closing the connection - the way how currrent sqlalchemy works
with it is that it leaves threads hanging. This is fixed
in sqlalchemy main - sqlalchemy/sqlalchemy#13039
but until they release it and until we have the sqlalchemy>=2.0.46,
we should upper-bind the aiosqlite
(cherry picked from commit 9b364ca)
Co-authored-by: Jarek Potiuk