Skip to content

feat(chronicleexporter): add exporter_unsplit_payload_size metric#488

Merged
dpaasman00 merged 1 commit into
mainfrom
chronicle-batch-size-metric
Jun 30, 2026
Merged

feat(chronicleexporter): add exporter_unsplit_payload_size metric#488
dpaasman00 merged 1 commit into
mainfrom
chronicle-batch-size-metric

Conversation

@dpaasman00

Copy link
Copy Markdown
Contributor

Proposed Change

Checklist
  • Changes are tested
  • CI has passed

Copy link
Copy Markdown
Contributor Author

@jsirianni jsirianni left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image

dpaasman00 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor Author

Merge activity

  • Jun 30, 3:30 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jun 30, 3:31 PM UTC: Graphite rebased this pull request as part of a merge.
  • Jun 30, 3:45 PM UTC: @dpaasman00 merged this pull request with Graphite.

Record the size in bytes of each export batch before it is split to
satisfy batch_request_size_limit_http and the one-log-type-per-request
grouping. Operators can compare this against batch_request_size_limit_http
to tune the batch processor's send_batch_size and avoid splitting.

Assisted-by: Claude Opus 4.8
@dpaasman00 dpaasman00 force-pushed the chronicle-batch-size-metric branch from df9059a to 6c4d8d5 Compare June 30, 2026 15:31
@dpaasman00 dpaasman00 merged commit 1a4ea2a into main Jun 30, 2026
30 checks passed
@dpaasman00 dpaasman00 deleted the chronicle-batch-size-metric branch June 30, 2026 15:45
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