Skip to content

[Event Hubs] Processor Event Dispatch Cancellation#22695

Merged
jsquire merged 1 commit intoAzure:mainfrom
jsquire:eventhubs/respect-cancellation
Jul 16, 2021
Merged

[Event Hubs] Processor Event Dispatch Cancellation#22695
jsquire merged 1 commit intoAzure:mainfrom
jsquire:eventhubs/respect-cancellation

Conversation

@jsquire
Copy link
Copy Markdown
Member

@jsquire jsquire commented Jul 16, 2021

Summary

The focus of these changes is to fix the loop responsible for dispatching events to the handler for processing so that it respects cancellation when the processor signals the associated token. Currently, the loop will dispatch all events in the batch to the handler, potentially causing duplicate processing and checkpoint interleaving.

@jsquire jsquire added Event Hubs Client This issue is related to a non-management package labels Jul 16, 2021
@jsquire jsquire added this to the [2021] August milestone Jul 16, 2021
@jsquire jsquire self-assigned this Jul 16, 2021
The focus of these changes is to fix the loop responsible for dispatching
events to the handler for processing so that it respects cancellation when
the processor signals the associated token.  Currently, the loop will dispatch
all events in the batch to the handler, potentially causing duplicate processing
and checkpoint interleaving.
@jsquire jsquire force-pushed the eventhubs/respect-cancellation branch from 6826289 to 8fadb27 Compare July 16, 2021 21:26
@jsquire jsquire merged commit 51212bb into Azure:main Jul 16, 2021
@jsquire jsquire deleted the eventhubs/respect-cancellation branch July 16, 2021 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Client This issue is related to a non-management package Event Hubs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants