| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
Shrink the vendored JsonWriter nesting stack from 32 entries to 8 entries. The stack still grows on demand for deeply nested payloads, while common SDK serialization avoids the larger initial array allocation. Co-Authored-By: Claude <noreply@anthropic.com>
📲 Install BuildsAndroid
|
Sorry, something went wrong.
Performance metrics 🚀
Baseline results on branch: perf/sdk-overhead-reductionStartup times
App size
Previous results on branch: perf/sdk-overhead-reduction-json-writer-stackStartup times
App size
|
Sorry, something went wrong.
There was a problem hiding this comment.
LGTM 👍
Sorry, something went wrong.
|
Cursor review |
Sorry, something went wrong.
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 6755ac1. Configure here.
Sorry, something went wrong.
…o perf/sdk-overhead-reduction-json-writer-stack # Conflicts: # CHANGELOG.md
…uction-json-writer-stack
| Back | FazBrowse Home | New Git URL |
PR Stack (SDK Overhead Reduction)
📜 Description
Reduce the initial nesting stack in the vendored JsonWriter from int[32] to int[8].
The stack still grows on demand through the existing push() logic when serializing deeply nested payloads.
💡 Motivation and Context
JsonWriter is created for envelope headers, item headers, and serialized payloads. Sentry SDK JSON is shallow in normal use, so the upstream GSON default stack size is larger than needed for the common path.
Starting with a smaller stack reduces per-writer allocation while preserving support for deeper JSON structures.
💚 How did you test it?
📝 Checklist
🔮 Next steps