[GitHub] [airflow] coinzerge commented on pull request #18622: Use seconds for schedule_delay metric

2021-10-13 Thread GitBox
coinzerge commented on pull request #18622: URL: https://github.com/apache/airflow/pull/18622#issuecomment-941660710 what's not right? total_seconds returns a float. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [airflow] coinzerge commented on pull request #18622: Use seconds for schedule_delay metric

2021-10-12 Thread GitBox
coinzerge commented on pull request #18622: URL: https://github.com/apache/airflow/pull/18622#issuecomment-941660710 what's not right? total_seconds returns a float. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [airflow] coinzerge commented on pull request #18622: Use seconds for schedule_delay metric

2021-10-08 Thread GitBox
coinzerge commented on pull request #18622: URL: https://github.com/apache/airflow/pull/18622#issuecomment-939180381 @uranusjr anything else i need to do to merge the PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [airflow] coinzerge commented on pull request #18622: Use seconds for schedule_delay metric

2021-10-07 Thread GitBox
coinzerge commented on pull request #18622: URL: https://github.com/apache/airflow/pull/18622#issuecomment-937142337 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsu

[GitHub] [airflow] coinzerge commented on pull request #18622: Use seconds for schedule_delay metric

2021-10-06 Thread GitBox
coinzerge commented on pull request #18622: URL: https://github.com/apache/airflow/pull/18622#issuecomment-937378491 added test. what do you mean by `we should also add annotations`? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [airflow] coinzerge commented on pull request #18622: Use seconds for schedule_delay metric

2021-10-06 Thread GitBox
coinzerge commented on pull request #18622: URL: https://github.com/apache/airflow/pull/18622#issuecomment-937142337 @uranusjr updated PR -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the spec