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

POC `startSpan` API

Open adinauer opened this issue 1 year ago • 3 comments

: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 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

adinauer avatar Sep 26 '24 08:09 adinauer

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

github-actions[bot] avatar Sep 26 '24 08:09 github-actions[bot]

FYI @markushi just so you're aware this exists, looks like we won't be using this atm though.

adinauer avatar Sep 26 '24 08:09 adinauer

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

github-actions[bot] avatar Sep 26 '24 08:09 github-actions[bot]

We're probably gonna implement this differently in the future (maybe next major)

adinauer avatar Feb 07 '25 08:02 adinauer