Skip to content

feat: trace batch writes#153

Merged
mag1cfrog merged 1 commit into
mainfrom
feat/145-batch-write-tracing
Jun 23, 2026
Merged

feat: trace batch writes#153
mag1cfrog merged 1 commit into
mainfrom
feat/145-batch-write-tracing

Conversation

@mag1cfrog

Copy link
Copy Markdown
Owner

Summary

  • add stable batch write trace names and phase names for batch schema validation, value conversion, direct encoding, and packet write failures
  • emit sanitized outer batch write telemetry for baseline, direct framed, and direct raw paths
  • record attempted batch ordinal, batch shape, empty flag, pre-failure stats, successful cumulative WriteStats, result status, diagnostic codes, and elapsed time
  • cover success, empty batches, validation failures, conversion failures, send failures, direct backends, and row-value redaction with unit fakes

Notes

This PR instruments the outer write_batch boundary. Direct raw per-packet and per-encoded-chunk telemetry remains scoped to #146.

Closes #145.

Validation

  • cargo fmt --all --check
  • cargo check --workspace
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace
  • git diff --check

@mag1cfrog mag1cfrog merged commit e7e15d3 into main Jun 23, 2026
2 checks passed
@mag1cfrog mag1cfrog deleted the feat/145-batch-write-tracing branch June 23, 2026 05:01
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.

Observability: trace BulkWriter batch writes across backends

1 participant