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

feat(profiling): new options class and property; lifecycle option #4856

Draft
wants to merge 3 commits into
base: armcknight/profiling/new-continuous-apis/2-new-start-stop-api
Choose a base branch
from

Conversation

armcknight
Copy link
Member

Implementing the new option for profile session lifecycle. Includes using the new swift namespacing pattern to house these new config options.

#skip-changelog; for #4853

Copy link

github-actions bot commented Feb 15, 2025

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 5d43904

Copy link

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1227.17 ms 1246.08 ms 18.92 ms
Size 22.32 KiB 821.01 KiB 798.70 KiB

@armcknight armcknight force-pushed the armcknight/profiling/new-continuous-apis/2-new-start-stop-api branch from a09d3e7 to 537c49d Compare February 18, 2025 07:04
    refactor remaining profiling code in tracer to file combining
    SentryProfiledTracerConcurrency and
    SentryCaptureTransactionWithProfile, and add functions to start
    and stop continuous profiler based on root span counts
@armcknight armcknight force-pushed the armcknight/profiling/new-continuous-apis/3-profile-lifecycle-option branch from 8128a4c to 5d43904 Compare February 18, 2025 07:05
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.

1 participant