-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Fixes 14528: skip statuses for tasks that are no longer present in Airflow DAG #14529
Fixes 14528: skip statuses for tasks that are no longer present in Airflow DAG #14529
Conversation
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
@mgorsk1 thanks for the contribution. |
The Python checkstyle failed. Please run You can install the pre-commit hooks with |
@mgorsk1 can you please fix style check failures, looks like test case is also failing because of the change |
The Python checkstyle failed. Please run You can install the pre-commit hooks with |
The Python checkstyle failed. Please run You can install the pre-commit hooks with |
Hey @harshach I've addressed whatever was brought up by CI, should be 👌 now |
|
Thanks for your contributions @mgorsk1 |
sure, always a pleasure 🤝🏼 |
Describe your changes:
Fixes #14528
I worked on filtering out task instances which are related to tasks no longer present in serialized dag object because fetching statuses for such tasks results in errors during ingestion.
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>