Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Calculate run duration with current time for running tasks in task and dag duration charts #38215

Closed
2 tasks done
tirkarthi opened this issue Mar 16, 2024 · 0 comments · Fixed by #38375
Closed
2 tasks done
Assignees
Labels
kind:feature Feature Requests

Comments

@tirkarthi
Copy link
Contributor

Description

Currently when a task instance is under execution there is start date but no end date. Hence in the charts run duration is calculated as 0 and the bar remains with 0 value. But in the grid the duration is calculated with current time - start date. The same change can be applied so that users know the task run duration and it also gives a growing bar which gives nice feedback with respect to average time.

We have a patch for this internally and will be happy to contribute.

Current bar remains zero till task finishes :

image

Proposed change to calculate run duration with current time :

image

Use case/motivation

It helps users know the task run duration visualized in the chart and it also gives a growing bar which gives nice feedback with respect to average time.

Related issues

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@tirkarthi tirkarthi added kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet labels Mar 16, 2024
@Taragolis Taragolis removed the needs-triage label for new issues that we didn't triage yet label Mar 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:feature Feature Requests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants