feat(logs): Do not send manual log origin by adinauer · Pull Request #4897 · getsentry/sentry-java
Navigation Menu
{{ message }}
getsentry / sentry-java Public
- Notifications You must be signed in to change notification settings
- Fork 467
Merged
Conversation
Copy link
Member
adinauer
commented
Nov 12, 2025
adinauer
commented
📜 Description
Do not send manual log origin to reduce log size.
💡 Motivation and Context
Fixes #4795
💚 How did you test it?
📝 Checklist
- I added GH Issue ID & Linear ID
- 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.
- I updated the wizard 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.
🔮 Next steps
adinauer
requested review from
lcian,
markushi,
romtsn and
stefanosiano
as code owners
Copy link
Contributor
| Messages | |
|---|---|
| 📖 | Do not forget to update Sentry-docs with your feature once the pull request gets approved. |
lcian approved these changes Nov 12, 2025
Copy link
Contributor
github-actions
bot
commented
Nov 12, 2025
github-actions bot commented
Nov 12, 2025Performance metrics 🚀
| Plain | With Sentry | Diff | |
|---|---|---|---|
| Startup time | 366.91 ms | 448.72 ms | 81.81 ms |
| Size | 1.58 MiB | 2.12 MiB | 555.26 KiB |
Baseline results on branch: main
Startup times
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| ee747ae | 400.46 ms | 423.61 ms | 23.15 ms |
| 9fbb112 | 361.43 ms | 427.57 ms | 66.14 ms |
| 9fbb112 | 359.71 ms | 421.85 ms | 62.14 ms |
| 33a08cc | 267.08 ms | 340.45 ms | 73.37 ms |
| 806307f | 357.85 ms | 424.64 ms | 66.79 ms |
| dcc6bbf | 382.58 ms | 462.13 ms | 79.54 ms |
| ee747ae | 405.43 ms | 485.70 ms | 80.28 ms |
| 1df7eb6 | 397.04 ms | 429.64 ms | 32.60 ms |
| 27d7cf8 | 397.90 ms | 498.65 ms | 100.75 ms |
| 3998a95 | 415.94 ms | 478.54 ms | 62.60 ms |
App size
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| ee747ae | 1.58 MiB | 2.10 MiB | 530.95 KiB |
| 9fbb112 | 1.58 MiB | 2.11 MiB | 539.18 KiB |
| 9fbb112 | 1.58 MiB | 2.11 MiB | 539.18 KiB |
| 33a08cc | 1.58 MiB | 2.12 MiB | 555.28 KiB |
| 806307f | 1.58 MiB | 2.10 MiB | 533.42 KiB |
| dcc6bbf | 1.58 MiB | 2.12 MiB | 553.10 KiB |
| ee747ae | 1.58 MiB | 2.10 MiB | 530.95 KiB |
| 1df7eb6 | 1.58 MiB | 2.10 MiB | 532.97 KiB |
| 27d7cf8 | 1.58 MiB | 2.12 MiB | 549.42 KiB |
| 3998a95 | 1.58 MiB | 2.10 MiB | 532.96 KiB |
adinauer
deleted the
feat/do-not-send-manual-log-origin
branch
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
2 participants