Skip to content

Remove global from openlineage provider#58868

Merged
jscheffl merged 2 commits intoapache:mainfrom
jscheffl:bugfix/remove-global-from-openlineage
Dec 1, 2025
Merged

Remove global from openlineage provider#58868
jscheffl merged 2 commits intoapache:mainfrom
jscheffl:bugfix/remove-global-from-openlineage

Conversation

@jscheffl
Copy link
Copy Markdown
Contributor

@jscheffl jscheffl commented Nov 30, 2025

Another small increment to remove global statements for PR #58116

This removes global statement in openlineage, a classsic thing where functools.cache() can help.

global is evil.

@potiuk potiuk force-pushed the bugfix/remove-global-from-openlineage branch from e549ac7 to a642860 Compare November 30, 2025 21:05
@jscheffl jscheffl merged commit b30bfa3 into apache:main Dec 1, 2025
82 checks passed
RoyLee1224 pushed a commit to RoyLee1224/airflow that referenced this pull request Dec 3, 2025
* Remove global from openlineage provider

* Fix pytests based on global variable
Copilot AI pushed a commit to jason810496/airflow that referenced this pull request Dec 5, 2025
* Remove global from openlineage provider

* Fix pytests based on global variable
itayweb pushed a commit to itayweb/airflow that referenced this pull request Dec 6, 2025
* Remove global from openlineage provider

* Fix pytests based on global variable
Subham-KRLX pushed a commit to Subham-KRLX/airflow that referenced this pull request Mar 4, 2026
* Remove global from openlineage provider

* Fix pytests based on global variable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants