integrations-core icon indicating copy to clipboard operation
integrations-core copied to clipboard

updates Airflow metric units

Open lgh2 opened this issue 1 year ago • 2 comments

What does this PR do?

Updates Airflow metric units to correspond to the values in the Airflow docs

Motivation

Customer request

Review checklist (to be filled by reviewers)

  • [ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • [ ] Changelog entries must be created for modifications to shipped code
  • [ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • [ ] If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

lgh2 avatar Jul 04 '24 00:07 lgh2

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.77%. Comparing base (1e1a1f7) to head (f0627fa). Report is 440 commits behind head on master.

Additional details and impacted files
Flag Coverage Δ
activemq ?
airflow 92.20% <ø> (?)
cassandra ?
hive ?
hivemq ?
hudi ?
ignite ?
jboss_wildfly ?
kafka ?
presto ?
solr ?

Flags with carried forward coverage won't be shown. Click here to find out more.

codecov[bot] avatar Jul 04 '24 00:07 codecov[bot]

It seems like the airflow documentation was updated. They now specify milliseconds as well, which is in line with our current doc.

dkirov-dd avatar Jul 18 '24 15:07 dkirov-dd