Skip to content

Conversation

@jscheffl
Copy link
Contributor

Another small increment to remove global statements for PR #58116

Here I was stumbling over a code block which seems to be very legacy and tracing back only one usage and by resolving, looks like a smaller piece of dead code. Trying to cut it out. Let's see if CI tells me there is need for more cleanup. Therefore DRAFT until CI is green

@jscheffl jscheffl added full tests needed We need to run full set of tests for this PR to merge all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs labels Nov 23, 2025
@jscheffl jscheffl force-pushed the bugfix/remove-global-from-task-instance-session branch from 34a3892 to d2e901a Compare November 23, 2025 20:38
@jscheffl jscheffl marked this pull request as ready for review November 23, 2025 22:02
@jscheffl jscheffl force-pushed the bugfix/remove-global-from-task-instance-session branch 2 times, most recently from 22d7d30 to 87e0b71 Compare November 25, 2025 21:19
@jscheffl jscheffl force-pushed the bugfix/remove-global-from-task-instance-session branch from 87e0b71 to 4f5081c Compare November 26, 2025 22:28
@potiuk potiuk merged commit bd1e5c0 into apache:main Nov 28, 2025
201 checks passed
@potiuk
Copy link
Member

potiuk commented Nov 28, 2025

FYI: I think I implemented it when I chased some error, where I did not know that we already use thread-local session

RoyLee1224 pushed a commit to RoyLee1224/airflow that referenced this pull request Dec 3, 2025
* Remove global from task instance session

* Fix pytest in standard provider for back-compat
Copilot AI pushed a commit to jason810496/airflow that referenced this pull request Dec 5, 2025
* Remove global from task instance session

* Fix pytest in standard provider for back-compat
itayweb pushed a commit to itayweb/airflow that referenced this pull request Dec 6, 2025
* Remove global from task instance session

* Fix pytest in standard provider for back-compat
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs area:CLI area:providers full tests needed We need to run full set of tests for this PR to merge provider:standard provider:weaviate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants