Skip to content
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

openlineage: adjust DefaultExtractor's on_failure detection for airflow 2.10 fix #41094

Merged
merged 1 commit into from
Aug 2, 2024

Conversation

kacpermuda
Copy link
Contributor

A followup to #41053, adjusting OpenLineage provider to work well after we change the place of the listener hook manager being called.

Should only be merged after #41053 is accepted and merged in current form.


^ 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.

@kacpermuda kacpermuda marked this pull request as ready for review August 1, 2024 12:12
@kacpermuda kacpermuda force-pushed the ol-fix-failure-default-extractor branch 2 times, most recently from 7bc6ebe to 0a27735 Compare August 1, 2024 12:16
@kacpermuda kacpermuda force-pushed the ol-fix-failure-default-extractor branch from 0a27735 to 0bdd6b9 Compare August 1, 2024 13:25
…low 2.10 fix

Signed-off-by: Kacper Muda <mudakacper@gmail.com>
@potiuk
Copy link
Member

potiuk commented Aug 1, 2024

LGTM. @eladkal ?

@eladkal eladkal merged commit 4d27069 into apache:main Aug 2, 2024
52 checks passed
@kacpermuda kacpermuda deleted the ol-fix-failure-default-extractor branch August 2, 2024 11:49
molcay pushed a commit to VladaZakharova/airflow that referenced this pull request Aug 19, 2024
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.

4 participants