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

Posted to commits@airflow.apache.org by "knil-sama (JIRA)" <ji...@apache.org> on 2018/01/26 00:23:01 UTC

[jira] [Created] (AIRFLOW-2034) mixup between %s and {} when using str.format

knil-sama created AIRFLOW-2034:
----------------------------------

             Summary: 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
             Fix For: 1.9.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)