-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
AIRFLOW-2787 Allow is_backfill to handle NULL DagRun.run_id #3629
Conversation
Test failure is unrelated. |
2cbdb4e
to
9dc5276
Compare
LGTM |
Do you have a pass travis CI link for your pr in your local repo? |
9dc5276
to
3c92c36
Compare
I have not seen a full set of passing tests yet but it is different CI targets that fail each time. The tests seem to be failing due to some sort of problem |
thanks @gwax . Could you rebase the pr with master? There was an issue with the master which fails the CI. It just get reverted. |
feel free to ping me once you rebase with master and we are good to go. |
3c92c36
to
cd2dfd5
Compare
@feng-tao rebased |
thanks @gwax . LGTM. |
Make sure you have checked all steps below.
JIRA
Description
is_backfill
will fail with aAttributeError: 'NoneType' object has no attribute 'startswith'
error when called on a DagRun with NULLrun_id
(allowed by model definition). This PR introduces a None check to prevent the crash.Tests
test_is_backfill
to check the NULL run_id caseCommits
Documentation
Code Quality
git diff upstream/master -u -- "*.py" | flake8 --diff