[ 
https://issues.apache.org/jira/browse/AIRFLOW-4399?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16827287#comment-16827287
 ] 

ASF subversion and git services commented on AIRFLOW-4399:
----------------------------------------------------------

Commit 05b827fbeefa422a107e2f4f6e2ebf72fe13ba35 in airflow's branch 
refs/heads/v1-10-test from Xiaodong
[ https://gitbox.apache.org/repos/asf?p=airflow.git;h=05b827f ]

[AIRFLOW-4399] Avoid duplicated os.path.isfile() check in models.dagbag (#5165)

The check removed in this commit always return True for sure,
because the same check has alreaddy been done earlier in this function
body

(cherry picked from commit af3090786b170baf32c75fbd03c5f277c3ffaef8)


> To avoid duplicated os.path.isfile(filepath) checks in models.dagbag
> --------------------------------------------------------------------
>
>                 Key: AIRFLOW-4399
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-4399
>             Project: Apache Airflow
>          Issue Type: Improvement
>          Components: models
>    Affects Versions: 1.10.3
>            Reporter: Xiaodong DENG
>            Assignee: Xiaodong DENG
>            Priority: Minor
>             Fix For: 1.10.4
>
>
> There are duplicated os.path.isfile(filepath) checks, among which the 2nd one 
> is not necessary.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to