FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

perf(core): SDK Overhead reduction for JVM by adinauer · Pull Request #5535 · getsentry/sentry-java · GitHub

perf(core): SDK Overhead reduction for JVM - #5535

Merged
adinauer merged 16 commits into
perf/sdk-overhead-reductionfrom
perf/sdk-overhead-jvm-v2
Jun 25, 2026
Merged

perf(core): SDK Overhead reduction for JVM#5535
adinauer merged 16 commits into
perf/sdk-overhead-reductionfrom
perf/sdk-overhead-jvm-v2

Conversation

adinauer commented Jun 12, 2026
edited
Loading

Copy link
Copy Markdown
Member

adinauer changed the title collection: SDK Overhead reduction for JVM perf(core): SDK Overhead reduction for JVM Jun 12, 2026

sentry Bot commented Jun 12, 2026
edited
Loading

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
SDK Size io.sentry.tests.size 8.45.0 (1) release

⚙️ sentry-android Build Distribution Settings

Avoid allocating and sorting a merged breadcrumb queue when only one component scope has breadcrumbs. This keeps the full merge path for multi-scope breadcrumbs and returns the default write scope queue when all scopes are empty.

Co-Authored-By: Claude <noreply@anthropic.com>
Use an explicit 512-character BufferedWriter buffer for envelope item and envelope serialization. This avoids allocating the oversized default char buffer for each short-lived serialization writer while preserving the existing OutputStreamWriter-based encoding path.

Co-Authored-By: Claude <noreply@anthropic.com>
adinauer and others added 3 commits June 18, 2026 14:55
Avoid creating temporary maps when applying scope and options tags or scope extras. The event setters already copy these maps, so this preserves snapshot semantics while reducing allocation overhead.

Co-Authored-By: Claude <noreply@anthropic.com>
adinauer closed this Jun 18, 2026
adinauer force-pushed the perf/sdk-overhead-jvm-v2 branch from 51c339c to 7c1a728 Compare June 18, 2026 12:58
…ies-v2

perf(core): [SDK Overhead reduction for JVM 1] Remove redundant event map copies
adinauer reopened this Jun 18, 2026

Copy link
Copy Markdown
Contributor

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 324.53 ms 363.54 ms 39.01 ms
Size 0 B 0 B 0 B

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
a416a65 333.78 ms 410.37 ms 76.59 ms
d15471f 304.55 ms 408.43 ms 103.87 ms
ca6b6d8 380.45 ms 460.38 ms 79.93 ms
c8125f3 397.65 ms 485.14 ms 87.49 ms
b936425 302.69 ms 372.86 ms 70.17 ms
ed33deb 334.19 ms 362.30 ms 28.11 ms
cf708bd 408.35 ms 458.98 ms 50.63 ms
eb95ded 317.51 ms 369.08 ms 51.57 ms
a5ab36f 320.47 ms 389.77 ms 69.30 ms
80fd6ad 321.06 ms 375.79 ms 54.73 ms

App size

Revision Plain With Sentry Diff
a416a65 1.58 MiB 2.12 MiB 555.26 KiB
d15471f 1.58 MiB 2.13 MiB 559.54 KiB
ca6b6d8 0 B 0 B 0 B
c8125f3 1.58 MiB 2.10 MiB 532.32 KiB
b936425 0 B 0 B 0 B
ed33deb 1.58 MiB 2.13 MiB 559.52 KiB
cf708bd 1.58 MiB 2.11 MiB 539.71 KiB
eb95ded 0 B 0 B 0 B
a5ab36f 1.58 MiB 2.12 MiB 555.26 KiB
80fd6ad 0 B 0 B 0 B

adinauer and others added 4 commits June 18, 2026 15:46
Avoid allocating merged collection copies when only one combined scope contains values. This extends the breadcrumbs optimization to tags, attributes, extras, and attachments while preserving merge behavior when multiple scopes contribute data.

Co-Authored-By: Claude <noreply@anthropic.com>
…umbs-v3

perf(core): [SDK Overhead reduction for JVM 2] Short-circuit combined scope breadcrumbs
adinauer changed the base branch from main to perf/sdk-overhead-reduction June 25, 2026 11:50
adinauer marked this pull request as ready for review June 25, 2026 12:09
adinauer requested a review from romtsn as a code owner June 25, 2026 12:09
adinauer merged commit 7e399ae into perf/sdk-overhead-reduction Jun 25, 2026
20 of 23 checks passed
adinauer deleted the perf/sdk-overhead-jvm-v2 branch June 25, 2026 12:09
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL