Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Processor bug #14280

Merged
merged 2 commits into from
Aug 17, 2020
Merged

Processor bug #14280

merged 2 commits into from
Aug 17, 2020

Conversation

JoshLove-msft
Copy link
Member

@JoshLove-msft JoshLove-msft commented Aug 17, 2020 •
edited
Loading

Previously, calling StopProcessingAsync simultaneously on multiple threads could result in a null ref exception. Now we avoid this, and explicitly throw an InvalidOperationException if StopProcessingAsync is called on an already stopped processor. This is consistent with how we handle calling StartProcessingAsync on an already started processor.

@ghost ghost added the Service Bus label Aug 17, 2020
@JoshLove-msft JoshLove-msft merged commit f9b7fcc into Azure:master Aug 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MiYanni MiYanni MiYanni approved these changes

@jsquire jsquire Awaiting requested review from jsquire

@ShivangiReja ShivangiReja Awaiting requested review from ShivangiReja

Assignees
No one assigned
Labels
Service Bus
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JoshLove-msft @MiYanni