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

fix(profiling): Show p75 only for suspect functions table #46140

Merged
merged 2 commits into from Mar 22, 2023

Conversation

Zylphrex
Copy link
Member

To align with suspect spans, we also only show p75 of function duration in suspect functions. This also reduces the width of the table.

Closes getsentry/team-profiling#196

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

To align with suspect spans, we also only show p75 of function duration in
suspect functions. This also reduces the width of the table.
@Zylphrex Zylphrex requested a review from a team as a code owner March 21, 2023 19:02
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 21, 2023
@Zylphrex Zylphrex merged commit 0159e2c into master Mar 22, 2023
49 checks passed
@Zylphrex Zylphrex deleted the txiao/fix/show-p75-only-for-suspect-functions-table branch March 22, 2023 14:45
@github-actions github-actions bot locked and limited conversation to collaborators Apr 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants