[jira] [Resolved] (AIRFLOW-1820) dagrun.dependency-check metric does not play nice with statsd

["Bolke de Bruin (JIRA)" <ji...@apache.org>]

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

Bolke de Bruin resolved AIRFLOW-1820.
-------------------------------------
       Resolution: Fixed
    Fix Version/s: 1.9.1

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

> dagrun.dependency-check metric does not play nice with statsd
> -------------------------------------------------------------
>
>                 Key: AIRFLOW-1820
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-1820
>             Project: Apache Airflow
>          Issue Type: Bug
>          Components: core
>            Reporter: William Pursell
>            Assignee: William Pursell
>            Priority: Minor
>             Fix For: 1.9.1
>
>   Original Estimate: 10m
>  Remaining Estimate: 10m
>
> The metric dagrun.dependency-check (https://github.com/apache/incubator-airflow/blob/master/airflow/models.py#L4594) is sent to statsd with spaces and colons in the metric name.  This is not ideal for statsd.  (I'm not really sure I see the point of putting a timestamp in a metric name at all!)  We should either do something like Stats.timing("dagrun.dependency-check.{}{:%Y%m%d_%H%M%S}". to use a nicer name, or just drop the timestamp completely.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)