Skip to content

Propagate cancellation tokens through ingestion and body storage pers… - #5742

Merged
johnsimons merged 2 commits into
masterfrom
john/cancel_part10
Aug 12, 2026
Merged

johnsimons merged 2 commits into
masterfrom
john/cancel_part10

Conversation

@johnsimons

Copy link
Copy Markdown
Member

This change continues the effort to consistently apply optional CancellationToken parameters to asynchronous methods within the IBodyStorage, IFailedErrorImportDataStore, and ingestion unit of work interfaces, along with their EFCore and RavenDB implementations.

Cancellation tokens are now propagated to underlying asynchronous database calls, ensuring proper cancellation of long-running operations. In the error processor, specific handling for OperationCanceledException was added to ensure message contexts are released during shutdown so that the ingestion loop can terminate rather than awaiting indefinitely.

@johnsimons
johnsimons requested a review from rbev August 11, 2026 23:16
@johnsimons johnsimons self-assigned this Aug 11, 2026
Base automatically changed from john/cancel_part9 to master August 12, 2026 03:30
…istence

This change continues the effort to consistently apply optional `CancellationToken` parameters to asynchronous methods within the `IBodyStorage`, `IFailedErrorImportDataStore`, and ingestion unit of work interfaces, along with their EFCore and RavenDB implementations.

Cancellation tokens are now propagated to underlying asynchronous database calls, ensuring proper cancellation of long-running operations. In the error processor, specific handling for `OperationCanceledException` was added to ensure message contexts are released during shutdown so that the ingestion loop can terminate rather than awaiting indefinitely.
…tore

Pass the cancellation token to the body storage fetch operation to ensure proper cancellation of database calls during failed message body retrieval.
@johnsimons
johnsimons merged commit 0f813d9 into master Aug 12, 2026
50 checks passed
@johnsimons
johnsimons deleted the john/cancel_part10 branch August 12, 2026 04:07
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