Skip to content

Comments

feat: add tracing for compaction and manifest#581

Merged
ethe merged 2 commits intomainfrom
obs/compaction-manifest-tracing
Feb 12, 2026
Merged

feat: add tracing for compaction and manifest#581
ethe merged 2 commits intomainfrom
obs/compaction-manifest-tracing

Conversation

@belveryin
Copy link
Collaborator

Summary:

  • Instrumented compaction planning, scheduling, execution, and manifest-apply flows with consistent tracing spans and structured events.
  • Added manifest CAS attempt/retry/failure events with stable, low-cardinality fields and table_id context.
  • Replaced ad-hoc stderr logging with tracing events and added spans at key async boundaries.

Testing:

  • cargo +nightly fmt --all -- --check
  • cargo clippy --workspace -- -D warnings
  • cargo build --verbose
  • cargo test --verbose
  • cargo llvm-cov --workspace --lcov --output-path target/coverage/lcov.info --fail-under-lines 80

@belveryin belveryin marked this pull request as ready for review February 5, 2026 16:44
@belveryin belveryin requested a review from ethe February 5, 2026 16:44
@ethe ethe merged commit a6a8c90 into main Feb 12, 2026
6 checks passed
@ethe ethe deleted the obs/compaction-manifest-tracing branch February 12, 2026 09:04
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