Skip to content

ref: Per-bucket limits, fix envelope chunking (8)#5595

Open
sentrivana wants to merge 4 commits intoivana/span-first-7-add-trace-decoratorfrom
ivana/span-first-8-bucket-based-limits-in-batcher
Open

ref: Per-bucket limits, fix envelope chunking (8)#5595
sentrivana wants to merge 4 commits intoivana/span-first-7-add-trace-decoratorfrom
ivana/span-first-8-bucket-based-limits-in-batcher

Conversation

@sentrivana
Copy link
Contributor

Description

  • correctly splice spans into multiple envelopes if a bucket contains more than 1000 spans from the same trace_id
  • add weight-based flushing (actual limit is 10 MB, we flush at 5 MB)
  • apply the weight and amount limits per trace_id rather than globally
  • add trace_id, parent_span_id to the transport format

Issues

Reminders

@sentrivana sentrivana marked this pull request as ready for review March 5, 2026 14:59
@sentrivana sentrivana requested a review from a team as a code owner March 5, 2026 14:59
@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Codecov Results 📊

1772 passed | ⏭️ 166 skipped | Total: 1938 | Pass Rate: 91.43% | Execution Time: 3m 24s

All tests are passing successfully.

❌ Patch coverage is 22.73%. Project has 11813 uncovered lines.

Files with missing lines (1)
File Patch % Lines
_span_batcher.py 32.86% ⚠️ 47 Missing

Generated by Codecov Action

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.

1 participant