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

utils: add flamegraph for heap profiles #10223

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

Conversation

erikgrinaker
Copy link
Contributor

Problem

Unlike CPU profiles, the /profile/heap endpoint can't automatically generate SVG flamegraphs. This requires the user to install and use pprof tooling, which is unnecessary and annoying.

Resolves #10203.

Summary of changes

Add format=svg for the /profile/heap route, and generate an SVG flamegraph using the inferno crate, similarly to what pprof-rs already does for CPU profiles.

@erikgrinaker erikgrinaker requested review from a team and arssher and removed request for a team December 22, 2024 17:12
Copy link

github-actions bot commented Dec 22, 2024

7095 tests run: 6798 passed, 0 failed, 297 skipped (full report)


Flaky tests (3)

Postgres 17

Postgres 15

Code coverage* (full report)

  • functions: 31.2% (8400 of 26887 functions)
  • lines: 47.9% (66689 of 139118 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
96908ef at 2024-12-22T22:37:53.271Z :recycle:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

utils: flamegraphs for pprof heap profiles
1 participant