sentry-java
sentry-java copied to clipboard
POC `startSpan` API
:scroll: Description
:bulb: Motivation and Context
: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
| Fails | |
|---|---|
| :no_entry_sign: | Please consider adding a changelog entry for the next release. |
| Messages | |
|---|---|
| :book: | Do not forget to update Sentry-docs with your feature once the pull request gets approved. |
Instructions and example for changelog
Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.
Example:
## Unreleased
- POC `startSpan` API ([#3724](https://github.com/getsentry/sentry-java/pull/3724))
If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description.
Generated by :no_entry_sign: dangerJS against abb5bf6fbca8c82a9e20e8ed74a65d5a59858bbd
FYI @markushi just so you're aware this exists, looks like we won't be using this atm though.
Performance metrics :rocket:
| Plain | With Sentry | Diff | |
|---|---|---|---|
| Startup time | 471.69 ms | 581.84 ms | 110.15 ms |
| Size | 1.70 MiB | 2.35 MiB | 667.29 KiB |
We're probably gonna implement this differently in the future (maybe next major)