Skip to content

Split worker trace job persistence helpers#283

Merged
yvette-carlisle merged 1 commit into
mainfrom
xy/split-worker-trace-persistence
Jun 29, 2026
Merged

Split worker trace job persistence helpers#283
yvette-carlisle merged 1 commit into
mainfrom
xy/split-worker-trace-persistence

Conversation

@yvette-carlisle

Copy link
Copy Markdown
Member

Summary

  • Split worker trace job SQL persistence insert helpers into worker/trace_jobs/persistence.rs.
  • Kept parent trace_jobs.rs focused on payload decoding, transaction orchestration, and commit order.
  • Preserved helper signatures, SQL, conflict handling, and narrow pub(super) visibility.

Validation

  • cargo make fmt-rust
  • cargo make check-rust
  • cargo make fmt-rust-check
  • git diff --check
  • cargo make lint-vstyle
  • cargo nextest run -p elf-worker --all-targets --all-features (6 passed)
  • cargo make lint-rust
  • cargo make test-rust (379 passed, 92 skipped)
  • cargo make check (379 passed, 92 skipped)
  • Docker-backed cargo make test-rust-integration (92 passed, 379 skipped)
  • Docker-backed cargo make check-trace-gate

Review

  • Scout selected trace persistence as a strong modularization boundary.
  • Skeptic passed with no findings after staged diff review.

…tence helpers after strict validation.","authority":"manual"}
@yvette-carlisle yvette-carlisle merged commit a16418a into main Jun 29, 2026
12 checks passed
@yvette-carlisle yvette-carlisle deleted the xy/split-worker-trace-persistence branch June 29, 2026 23: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.

1 participant