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.

Sign up for GitHub

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

Jump to bottom

trace command print the percentage of time cost #2156

Merged
merged 2 commits into from Apr 22, 2022
Merged

trace command print the percentage of time cost #2156

merged 2 commits into from Apr 22, 2022

Conversation

yqxzy
Copy link
Contributor

@yqxzy yqxzy commented Apr 21, 2022

fix issue#1528

@yqxzy
Update TraceView.java
c7d459c 
fix issue#1528
@CLAassistant
Copy link

CLAassistant commented Apr 21, 2022 •
edited

CLA assistant check
All committers have signed the CLA.

@yqxzy yqxzy changed the title Update TraceView.java fix the issue#1528 Apr 22, 2022
@hengyunabc hengyunabc added this to the 3.6.1 milestone Apr 22, 2022
@yqxzy
Update TraceView.java
037af65 
Sorry, I have fixed the problem by adding another line for the variable, please check again. Thank you.
@hengyunabc hengyunabc merged commit 620bdf6 into alibaba:master Apr 22, 2022
hengyunabc added a commit that referenced this pull request Apr 22, 2022
@hengyunabc
polish #2156
1f1cfd0
@hengyunabc
Copy link
Collaborator

改进了下，把 %放到最前面，比较直观。 另外，计算多次执行过的node，要用 total。

image

@hengyunabc hengyunabc changed the title fix the issue#1528 trace command print the percentage of time cost Apr 24, 2022
@hengyunabc hengyunabc mentioned this pull request Apr 24, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.6.1
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@yqxzy @CLAassistant @hengyunabc