Fix import of DummyStatsLogger for airflow >= v2.6.0

newrelic / newrelic-airflow-plugin

Send airflow metrics to New Relic!

Apache License 2.0

25 stars 19 forks source link

Fix import of DummyStatsLogger for airflow >= v2.6.0 #42

Closed shuvaevv closed 1 year ago

shuvaevv commented 1 year ago

Fix the DummyStatsLogger import.

Closes: #40

This bug appeared because DummyStatsLogger was renamed to NullStatsLogger in airflow 2.6.0.

The import of the DummyStatsLogger is try-excepted, so the DummyStatsLogger class is None. It leads to the AttributeError: 'NoneType' object has no attribute 'incr'.

CLAassistant commented 1 year ago

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: shuvaevv
:x: Valentin Shuvaev

Valentin Shuvaev seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}