You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2019/01/06 03:36:35 UTC

[GitHub] XD-DENG commented on issue #4445: [AIRFLOW-3635] Fix incorrect logic in detele_dag (introduced in PR#4406)

XD-DENG commented on issue #4445: [AIRFLOW-3635] Fix incorrect logic in detele_dag (introduced in PR#4406)
URL: https://github.com/apache/airflow/pull/4445#issuecomment-451712399
 
 
   Hi @kaxil , thanks for fixing the Travis CI. As you shared, there are about **`8 FAILED, 2 errors`** currently in the latest commit in master branch. I found that some of them were due to PR #4406 . An obvious logic error was found in it.
   
   With the fix in this PR, we reduce the fails and errors to **`3 FAILED, 1 error`** (the rest are due to other reasons).
   
   CC @ffinfo who authored #4406 , and @Fokko who reviewed. Please check this PR and confirm if the fix is making sense to you.
   
   Thanks.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services