Skip to content

fix(appStart): Attach App Start spans to the first created not the first processed root span (#4618)#4644

Merged
krystofwoldrich merged 4 commits into
hotfix/6.9.1from
pick-fix-app-start-first-tx
Mar 11, 2025
Merged

fix(appStart): Attach App Start spans to the first created not the first processed root span (#4618)#4644
krystofwoldrich merged 4 commits into
hotfix/6.9.1from
pick-fix-app-start-first-tx

Conversation

@krystofwoldrich

Copy link
Copy Markdown
Contributor

Cherry pick #4618 for 6.9.1 hotfix

Krystof Woldrich and others added 3 commits March 11, 2025 13:01
Updates the Android SDK dependency to the latest patch version, which includes potential bug fixes and improvements for measuring cold and warm starts.
@krystofwoldrich krystofwoldrich changed the title chore(android): Bump Sentry Android SDK to v7.22.1 fix(appStart): Attach App Start spans to the first created not the first processed root span (#4618) Mar 11, 2025
@github-actions

github-actions Bot commented Mar 11, 2025

Copy link
Copy Markdown
Contributor

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1215.42 ms 1223.72 ms 8.30 ms
Size 3.19 MiB 4.32 MiB 1.13 MiB

@github-actions

github-actions Bot commented Mar 11, 2025

Copy link
Copy Markdown
Contributor

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1214.84 ms 1216.42 ms 1.58 ms
Size 2.63 MiB 3.75 MiB 1.12 MiB

@antonis antonis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The cherrypicked code LGTM 🙇

Base automatically changed from bump-to-android-7-22-1 to hotfix/6.9.1 March 11, 2025 13:50
@krystofwoldrich krystofwoldrich merged commit 4295cdc into hotfix/6.9.1 Mar 11, 2025
@krystofwoldrich krystofwoldrich deleted the pick-fix-app-start-first-tx branch March 11, 2025 13:50
@github-actions

Copy link
Copy Markdown
Contributor
Fails
🚫 Please consider adding a changelog entry for the next release.

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

- Attach App Start spans to the first created not the first processed root span (#4618) ([#4644](https://github.com/getsentry/sentry-react-native/pull/4644))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description.

Generated by 🚫 dangerJS against c061093

@github-actions

Copy link
Copy Markdown
Contributor

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 476.39 ms 481.21 ms 4.83 ms
Size 17.75 MiB 20.12 MiB 2.37 MiB

@github-actions

Copy link
Copy Markdown
Contributor

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 398.87 ms 383.74 ms -15.13 ms
Size 7.15 MiB 8.39 MiB 1.23 MiB

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.

2 participants