v1.0.37 - #99
Conversation
feat(outbox): carry the drop's trace identity and re-join Outbox.Proc…
Reviewer's GuideAdds end-to-end outbox trace continuity by persisting the publishing activity’s trace identity and restoring it on Outbox.Process spans, with worker-loop links and backward-compatible fallbacks for existing or untraced rows. Sequence diagram for outbox trace continuitysequenceDiagram
participant Publisher
participant EventBus
participant EfCoreOutboxStore
participant OutboxProcessor
participant Broker
Publisher->>EventBus: Publish
EventBus->>EfCoreOutboxStore: StoreAsync
EfCoreOutboxStore->>EfCoreOutboxStore: Persist TraceParent and TraceState
EfCoreOutboxStore->>EventBus: SetTag outbox.message_id
OutboxProcessor->>OutboxProcessor: StartActivity Outbox.Process
OutboxProcessor->>OutboxProcessor: ActivityContext.TryParse
OutboxProcessor->>Broker: Publish event
alt Trace identity exists
OutboxProcessor->>OutboxProcessor: Parent Outbox.Process on origin and link worker loop
else No trace identity
OutboxProcessor->>OutboxProcessor: Parent Outbox.Process on worker loop
end
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Hey - I've reviewed your changes and they look great!
Sourcery assessment
Needs a human reviewer. If the trace context handling is wrong, Outbox.Process spans could be mis-parented or produce misleading distributed traces, and incorrect trace metadata would remain on already-written outbox rows after a revert. The impact is bounded and the rows can be cleaned or reprocessed; no business data, access decision, or irreversible action is changed.
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 26 |
| Duplication | 4 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
|



Summary by Sourcery
Maintain distributed-trace continuity across outbox persistence and processing.
New Features:
Enhancements:
Tests: