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.

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

Unblock message pump while receiving message #6804

Conversation

alexander-rakov
Copy link
Contributor

Previous version of the code blocked further task processing with more
Tasks being created while message was being received up until the point
when message processing actually started.

With switch from await to ContinueWith execution of the rest of the
method is continued asynchronously allowing MaxConcurrentCalls to be
higher than number of logical processors on the target machine.

Previous version of the code blocked further task processing with more
Tasks being created while message was being received up until the point
when message processing actually started.

With switch from await to ContinueWith execution of the rest of the
method is continued asynchronously allowing `MaxConcurrentCalls` to be
higher than number of logical processors on the target machine.
@alexander-rakov alexander-rakov force-pushed the servicebus-fix-messagereceivepump-async branch from 9c36949 to 4e2f7e5 Compare July 2, 2019 23:34
@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@alexander-rakov
Copy link
Contributor Author

Looks like net - mgmt - ci check failures all related to System.InvalidOperationException : 'SERVICE_BUS_CONNECTION_STRING' environment variable was not found!. I verified that if I run it locally both servicebus and mgmt tests succeeds (deployed test Azure Service Bus under my Azure subscription).

@jsquire jsquire requested review from nemakam and vinaysurya July 8, 2019 14:59
@jsquire jsquire requested a review from binzywu July 8, 2019 14:59
@jsquire jsquire added Client This issue points to a problem in the data-plane of the library. Service Bus labels Jul 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Client This issue points to a problem in the data-plane of the library. Service Bus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants