Skip to content

fix(core): Record byte discards for event processor drops#5718

Merged
adinauer merged 2 commits into
mainfrom
fix/event-processor-byte-discards
Jul 3, 2026
Merged

fix(core): Record byte discards for event processor drops#5718
adinauer merged 2 commits into
mainfrom
fix/event-processor-byte-discards

Conversation

@adinauer

@adinauer adinauer commented Jul 3, 2026

Copy link
Copy Markdown
Member

📜 Description

Record byte-count client report outcomes when event processors drop log or trace metric events.

This adds LogByte and TraceMetricByte discarded-event entries alongside the existing LogItem and TraceMetric count entries for EventProcessor drops.

💡 Motivation and Context

Before this change, before-send and queue-overflow discard paths tracked both item count and byte count, but EventProcessor drops only tracked item count. This made byte-level client report accounting incomplete for logs and trace metrics.

💚 How did you test it?

  • ./gradlew :sentry:test --tests "io.sentry.SentryClientTest"
  • ./gradlew spotlessApply apiDump

📝 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 sendDefaultPII is 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.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.

🔮 Next steps

adinauer and others added 2 commits July 3, 2026 14:22
Track log and trace metric byte counts when event processors drop log or metric events. This keeps EventProcessor discard accounting aligned with before-send and queue-overflow paths.

Co-Authored-By: Claude <noreply@anthropic.com>
@sentry

sentry Bot commented Jul 3, 2026

Copy link
Copy Markdown

📲 Install Builds

Android

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

⚙️ sentry-android Build Distribution Settings

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 338.04 ms 400.31 ms 62.27 ms
Size 0 B 0 B 0 B

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
abfcc92 309.54 ms 380.32 ms 70.78 ms
4e3e79d 328.10 ms 395.64 ms 67.54 ms
8c1fb22 316.62 ms 352.78 ms 36.16 ms
b67bb28 307.59 ms 341.24 ms 33.65 ms
d15471f 361.89 ms 378.07 ms 16.18 ms
22f4345 313.52 ms 364.96 ms 51.44 ms
6727e14 337.22 ms 373.94 ms 36.71 ms
fc5ccaf 322.49 ms 405.25 ms 82.76 ms
22f4345 314.79 ms 375.02 ms 60.23 ms
2195398 319.02 ms 342.38 ms 23.36 ms

App size

Revision Plain With Sentry Diff
abfcc92 1.58 MiB 2.13 MiB 557.31 KiB
4e3e79d 0 B 0 B 0 B
8c1fb22 0 B 0 B 0 B
b67bb28 0 B 0 B 0 B
d15471f 1.58 MiB 2.13 MiB 559.54 KiB
22f4345 1.58 MiB 2.29 MiB 719.83 KiB
6727e14 1.58 MiB 2.28 MiB 718.64 KiB
fc5ccaf 1.58 MiB 2.13 MiB 557.54 KiB
22f4345 1.58 MiB 2.29 MiB 719.83 KiB
2195398 0 B 0 B 0 B

@adinauer adinauer marked this pull request as ready for review July 3, 2026 12:55
@adinauer adinauer merged commit 44d18f5 into main Jul 3, 2026
71 checks passed
@adinauer adinauer deleted the fix/event-processor-byte-discards branch July 3, 2026 13:25
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