airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bolke de Bruin (JIRA)" <j...@apache.org>
Subject [jira] [Resolved] (AIRFLOW-2034) mixup between %s and {} when using str.format
Date Sun, 25 Feb 2018 11:34:00 GMT

     [ https://issues.apache.org/jira/browse/AIRFLOW-2034?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Bolke de Bruin resolved AIRFLOW-2034.
-------------------------------------
       Resolution: Fixed
    Fix Version/s:     (was: 1.9.0)
                   1.10.0

Issue resolved by pull request #2976
[https://github.com/apache/incubator-airflow/pull/2976]

> mixup between %s and {} when using str.format
> ---------------------------------------------
>
>                 Key: AIRFLOW-2034
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-2034
>             Project: Apache Airflow
>          Issue Type: Bug
>          Components: hooks
>    Affects Versions: 1.9.0
>            Reporter: knil-sama
>            Assignee: knil-sama
>            Priority: Trivial
>              Labels: easyfix
>             Fix For: 1.10.0
>
>
> Convention is to use .format for string formating outside logging, else use lazy formation
>  See comment in related issue
>  #[https://github.com/apache/incubator-airflow/pull/2823/files]
> But some code didn't implement it correctly.
> Problematic cases can be identified using following command line
> {{grep -r '%s'./* | grep '\.format('}}



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

Mime
View raw message