-
Notifications
You must be signed in to change notification settings - Fork 14.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] Remove old lineage stuff #45260
base: main
Are you sure you want to change the base?
Conversation
When attempting to remove the lineage logic from the core module, I noticed that it causes failures in tests related to OpenLineage listener capturing hook-level lineage (#41482). For example, removing airflow/airflow/models/baseoperator.py Lines 705 to 739 in 0efd9e6
Results in the following test failure:
It seems OpenLineage is still coupled with the lineage module and might need to be moved to After some experimentation, implementing an I’m not sure if this is a suitable long-term solution for maintaining OpenLineage compatibility while cleaning up the lineage module ? |
- allowed_deserialization_classes in unit_tests config - import path in pytest_plugin
7975d5a
to
727a6c1
Compare
closes: #44983
^ 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 newsfragments.