Remove auto lineage from Airflow #48421
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
uranusjr
requested review from
amoghrajesh,
ashb,
kaxil and
mobuchowski
as code owners
boring-cyborg
bot
added
area:lineage
area:providers
area:task-sdk
provider:openlineage
uranusjr
force-pushed
the
remove-lineage
branch
2 times, most recently
from
900d908 to
e81bde2
Compare
Member
I'm just curious, but why? |
Member
Author
|
From what I understand, the |
uranusjr
force-pushed
the
remove-lineage
branch
from
e81bde2 to
60e2357
Compare
kaxil
approved these changes
Mar 27, 2025
mobuchowski
reviewed
Mar 27, 2025
providers/papermill/src/airflow/providers/papermill/operators/papermill.py
Show resolved
Hide resolved
mobuchowski
reviewed
Mar 27, 2025
providers/papermill/src/airflow/providers/papermill/operators/papermill.py
Show resolved
Hide resolved
Contributor
|
@jedcunningham totally separate and different feature. Maybe it's unfortunate that it uses the same directory. |
mobuchowski
approved these changes
Mar 27, 2025
Contributor
mobuchowski
left a comment
mobuchowski
left a comment
There was a problem hiding this comment.
@uranusjr looks good outside of the PapermillOperator changes - we should keep Airflow 2 compatibility there
Member
Author
|
I’ve restored the functionality in PapermillOperator on 2.x for now. |
amoghrajesh
approved these changes
Mar 28, 2025
potiuk
approved these changes
Mar 31, 2025
This never really caught up, and we have other ways to do lineage now. The airflow.lineage.hook module is still available.
uranusjr
force-pushed
the
remove-lineage
branch
from
fa718c3 to
e897ae8
Compare
shubham-pyc
pushed a commit
to shubham-pyc/airflow
that referenced
this pull request
Apr 2, 2025
This never really caught up, and we have other ways to do lineage now. The airflow.lineage.hook module is still available. Usage in Papermill is also kept since we want it to work on 2.x for now.
nailo2c
pushed a commit
to nailo2c/airflow
that referenced
this pull request
Apr 4, 2025
This never really caught up, and we have other ways to do lineage now. The airflow.lineage.hook module is still available. Usage in Papermill is also kept since we want it to work on 2.x for now.
simonprydden
pushed a commit
to simonprydden/airflow
that referenced
this pull request
Apr 8, 2025
This never really caught up, and we have other ways to do lineage now. The airflow.lineage.hook module is still available. Usage in Papermill is also kept since we want it to work on 2.x for now.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This never really caught up, and we have other ways to do lineage now.
The airflow.lineage.hook module is still available.
Close #48388.