-
Notifications
You must be signed in to change notification settings - Fork 376
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
[Gluten-986] Rename metrics name from wall nanos to totaltime to xxx. #1034
Conversation
Thanks for opening a pull request! Could you open an issue for this pull request on Github Issues? https://github.com/oap-project/gluten/issues Then could you also rename commit message and pull request title in the following format?
See also: |
The CI failure during step |
@Yohahaha Thank you. We find cpu count is also useful to us, and can be used to calculte the avg time. Could you help add that back? |
Sure, submit commit in this pr or open another pr? How do you calculate avg time? cpu_time / cpu_count or wall_time / cpu_count? @rui-mo |
@Yohahaha Both work for us. Thank you.
I think wall_time is more useful to us. |
Got it. It looks like |
OK, let's remove cpu time, use wall time. |
1bcc7f3
to
b9335b0
Compare
…apache#1034) * Rename wall nanos to totaltime to xxx. * Replace cpu time to cpu count. --------- Co-authored-by: yangchuan <yangchuan.zy@alibaba-inc.com>
Mentioned in #987