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

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/12/14 12:09:09 UTC

[GitHub] [airflow] kaxil commented on pull request #20112: Add td_format function and apply to Views duration column for better human readability

kaxil commented on pull request #20112:
URL: https://github.com/apache/airflow/pull/20112#issuecomment-993476639


   > @bbovenzi when you recommend that format, `01:00:02` that is what it currently is. That is how the timedeltas look right now if I'm not mistaken. Unless you are looking for something that goes 5d:4h:32m:45s is that what you're recommending?
   > 
   > I'd like to get this in as soon as possible this week so if you don't mind quickly iterating with me to complete the ask.
   
   `01:00:02` vs '1 hour, 2 seconds' (this is what the PR does currently, no?). I think that is what Brent is suggesting.
   
   I am fine with either, I don't have a strong opinion on it.


-- 
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 unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org