sentry-java
sentry-java copied to clipboard
Handle App Start Continuous Profiling v8 (p4)
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
sendDefaultPIIis 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
| 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
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 |