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

Format elapsed time and critical path time #22331

Closed
marekkrk opened this issue May 11, 2024 · 4 comments
Closed

Format elapsed time and critical path time #22331

marekkrk opened this issue May 11, 2024 · 4 comments

Comments

@marekkrk
Copy link

Description of the feature request:

Bazel currently formats elapsed build time and critical path time in seconds. Example:
Elapsed time: 23120.370s, Critical Path: 131.10s

It would be good to use human friendly formatting, for example:
Elapsed time: 6h 25m 20.370s, Critical Path: 2m 11.10s

Which category does this issue belong to?

CLI

What underlying problem are you trying to solve with this feature?

Usability

Which operating system are you running Bazel on?

Linux

What is the output of bazel info release?

release 7.1.2

If bazel info release returns development version or (@non-git), tell us how you built Bazel.

No response

What's the output of git remote get-url origin; git rev-parse HEAD ?

No response

Have you found anything relevant by searching the web?

No response

Any other information, logs, or outputs that you want to share?

No response

@brentleyjones
Copy link
Contributor

@katre most recently (iirc) tried to address this.

@nikhilkalige
Copy link
Contributor

Similar request #17137

@katre
Copy link
Member

katre commented May 13, 2024

I took a look at this a few months ago and didn't make much progress. The underlying issue was mostly down to code complexity: in various places we're using Duration, int, long, and double to hold data that is either Duration, seconds, or milliseconds, and trying to resolve it all was too complex.

I'd love to see better progress here, so don't let me discourage you, but there aren't any quick fixes to be had, unfortunately.

@oquenchil
Copy link
Contributor

Duplicate of #17137

@oquenchil oquenchil marked this as a duplicate of #17137 May 14, 2024
@oquenchil oquenchil closed this as not planned Won't fix, can't repro, duplicate, stale May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants