Skip to content

Conversation

@jscheffl
Copy link
Contributor

@jscheffl jscheffl commented Nov 13, 2025

Another small increment to remove globalstatements for PR #58116

Replace global variable with a cache()
Which even (after adusting pytests) also saves some lines of code :-D

@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 13, 2025
@jscheffl jscheffl closed this Nov 13, 2025
@jscheffl jscheffl reopened this Nov 13, 2025
@jscheffl jscheffl force-pushed the bugfix/remove-global-from-lineage-hook branch 2 times, most recently from 009648d to 38a0bd7 Compare November 15, 2025 20:32
@jscheffl jscheffl force-pushed the bugfix/remove-global-from-lineage-hook branch 2 times, most recently from d3c5d70 to feffcf1 Compare November 22, 2025 21:15
@jscheffl jscheffl force-pushed the bugfix/remove-global-from-lineage-hook branch from feffcf1 to df7aad7 Compare November 23, 2025 16:14
@potiuk potiuk merged commit 7a51b05 into apache:main Nov 24, 2025
202 checks passed
Copilot AI pushed a commit to jason810496/airflow that referenced this pull request Dec 5, 2025
* Remove global from lineage.hook

* Fix pytest

* Prevent fail on back.compat

* Remove old fixtures as available in pytest-plugin

* Include backcompat in pytest plugin

* Fix openlineage pytest
itayweb pushed a commit to itayweb/airflow that referenced this pull request Dec 6, 2025
* Remove global from lineage.hook

* Fix pytest

* Prevent fail on back.compat

* Remove old fixtures as available in pytest-plugin

* Include backcompat in pytest plugin

* Fix openlineage pytest
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:lineage full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants