sentry-java icon indicating copy to clipboard operation
sentry-java copied to clipboard

Handle App Start Continuous Profiling v8 (p4)

Open stefanosiano opened this issue 1 year ago • 1 comments
trafficstars

:scroll: Description

create app start continuous profiler instead of transaction profiler, based on config updated SentryAppStartProfilingOptions with isContinuousProfilingEnabled flag updated SentryOptions with isContinuousProfilingEnabled() method

#skip-changelog

:bulb: Motivation and Context

Handle App Start Continuous Profiler Part 4 of https://github.com/getsentry/sentry-java/pull/3710

:green_heart: How did you test it?

:pencil: Checklist

  • [ ] I reviewed the submitted code.
  • [ ] I added tests to verify the changes.
  • [ ] No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • [ ] I updated the docs if needed.
  • [ ] Review from the native team if needed.
  • [ ] No breaking change or entry added to the changelog.
  • [ ] No breaking change for hybrid SDKs or communicated to hybrid SDKs.

:crystal_ball: Next steps

stefanosiano avatar Sep 30 '24 10:09 stefanosiano

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

Generated by :no_entry_sign: dangerJS against 540e276d8530811bad68f8e440de40b2a442bde1

github-actions[bot] avatar Sep 30 '24 10:09 github-actions[bot]

Performance metrics :rocket:

  Plain With Sentry Diff
Startup time 409.29 ms 423.43 ms 14.14 ms
Size 1.65 MiB 2.31 MiB 677.72 KiB

Previous results on branch: feat/continuous-profiling-part4

Startup times

Revision Plain With Sentry Diff
b1083a7fdff51e337037bc841a76aa7239b18094 427.67 ms 483.64 ms 55.96 ms

App size

Revision Plain With Sentry Diff
b1083a7fdff51e337037bc841a76aa7239b18094 1.65 MiB 2.31 MiB 676.46 KiB

github-actions[bot] avatar Nov 13 '24 16:11 github-actions[bot]