RUM-16712: Lower Continuous Profiling sampling frequency to 101Hz#3531
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## feature/continuous-profiling #3531 +/- ##
================================================================
- Coverage 72.41% 72.39% -0.02%
================================================================
Files 980 980
Lines 36268 36274 +6
Branches 6037 6038 +1
================================================================
- Hits 26261 26259 -2
- Misses 8361 8370 +9
+ Partials 1646 1645 -1
🚀 New features to boost your workflow:
|
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep it up! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
kikoveiga
left a comment
There was a problem hiding this comment.
Just an idea I've had for some time: all over the codebase we use samplingRate and sampleRate. We could think about aligning these terms and deciding to just keep one of them.
|
we should use |
What does this PR do?
Reduces the stack sampling rate for continuous profiling sessions from 201Hz to 101Hz to lower the profile density and CPU overhead. App launch profiling retains the higher 201Hz rate. The active rate is now tracked per-session and reported accurately in telemetry.
Review checklist (to be filled by reviewers)