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

feat: support profile result filter #8128

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat: support profile result filter #8128

wants to merge 1 commit into from

Conversation

lzf575
Copy link
Contributor

@lzf575 lzf575 commented Sep 12, 2024

支持过滤 profile的数据返回

  • 支持 total-value-percent：默认值 0.1 和 depth 默认值 12

  • total-value 占比小于 0.1% 且 节点层级大于 12 的才忽略

  • 目前对于off-cpu 可以减少 50%的显示数据，提高近50%性能

  • 如果 把 total-value-percent 调大， depth 调小，可丢弃更多节点，加载更快

This PR is for:

  • Server

sharang
sharang reviewed Sep 13, 2024
View reviewed changes
server/server.yaml Outdated Show resolved Hide resolved
@lzf575 lzf575 force-pushed the feature-test-profile-perf branch 4 times, most recently from 4bf1db1 to 72eb71c Compare September 13, 2024 02:41
@lzf575 lzf575 requested a review from sharang September 13, 2024 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sharang sharang Awaiting requested review from sharang

At least 1 approving review is required to merge this pull request.

Assignees

@sharang sharang

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lzf575 @sharang