Skip to content

feat(profiling): Pass thread id to flamegraph generation#74846

Merged
Zylphrex merged 2 commits into
masterfrom
txiao/feat/pass-thread-id-to-flamegraph-generation
Jul 25, 2024
Merged

feat(profiling): Pass thread id to flamegraph generation#74846
Zylphrex merged 2 commits into
masterfrom
txiao/feat/pass-thread-id-to-flamegraph-generation

Conversation

@Zylphrex

@Zylphrex Zylphrex commented Jul 24, 2024

Copy link
Copy Markdown
Member

To generate a more accurate flamegraph, we want samples from the active thread.

Depends on getsentry/snuba#6138

To generate a more accurate flamegraph, we want samples from the active thread.
@Zylphrex Zylphrex requested a review from a team as a code owner July 24, 2024 15:03
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 24, 2024
@codecov

codecov Bot commented Jul 24, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.13%. Comparing base (0c81115) to head (3bf4c1f).
Report is 4 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #74846       +/-   ##
===========================================
+ Coverage   54.34%   78.13%   +23.78%     
===========================================
  Files        5679     6740     +1061     
  Lines      239161   300847    +61686     
  Branches    40977    51745    +10768     
===========================================
+ Hits       129982   235052   +105070     
+ Misses     107732    59456    -48276     
- Partials     1447     6339     +4892     
Files Coverage Δ
src/sentry/profiles/flamegraph.py 82.80% <100.00%> (ø)
src/sentry/snuba/events.py 100.00% <100.00%> (ø)

... and 2476 files with indirect coverage changes

@Zylphrex Zylphrex merged commit a705fd4 into master Jul 25, 2024
@Zylphrex Zylphrex deleted the txiao/feat/pass-thread-id-to-flamegraph-generation branch July 25, 2024 18:53
Christinarlong pushed a commit that referenced this pull request Jul 26, 2024
To generate a more accurate flamegraph, we want samples from the active
thread.

Depends on getsentry/snuba#6138
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants