bbovenzi commented on code in PR #38374:
URL: https://github.com/apache/airflow/pull/38374#discussion_r1537418689
##########
airflow/www/static/js/dag/details/task/TaskDuration.tsx:
##########
@@ -179,7 +170,18 @@ const TaskDuration = () => {
// @ts-ignore
dataset: {
dimensions: ["runId", "queuedDurationUnit", "runDurationUnit"],
- source: durations,
+ source: durations.map((duration) => {
+ if (duration) {
+ const durationInSeconds = duration as TaskInstanceDuration;
Review Comment:
Yes, my bad. I remember now why we changed that. We needed empty columns in
case there was no task instance for a particular dag run. Let's leave you work
as-is then
--
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-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
