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

[Messaging Clients] README Concept Updates #18804

Merged
merged 1 commit into from
Feb 17, 2021

Conversation

jsquire
Copy link
Member

@jsquire jsquire commented Feb 16, 2021

Summary

The focus of these changes is to add content for lifetime, thread-safety, and additional concepts specific to the messaging clients. The content and format is intended to match the common blocks used by the HTTP-based service clients as closely as reasonable.

Last Upstream Rebase

Tuesday, February 16, 2021, 12:00pm (EST)

References and Related

@jsquire jsquire added Service Bus Event Hubs Client This issue points to a problem in the data-plane of the library. Docs labels Feb 16, 2021
@jsquire jsquire added this to the [2021] March milestone Feb 16, 2021
@jsquire jsquire self-assigned this Feb 16, 2021
sdk/servicebus/Azure.Messaging.ServiceBus/README.md Outdated Show resolved Hide resolved
sdk/servicebus/Azure.Messaging.ServiceBus/README.md Outdated Show resolved Hide resolved
sdk/servicebus/Azure.Messaging.ServiceBus/README.md Outdated Show resolved Hide resolved
@jsquire jsquire force-pushed the messaging/readme-updates branch 3 times, most recently from 8d2f7af to fcfd15c Compare February 16, 2021 20:33
Copy link
Member

@JoshLove-msft JoshLove-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

The focus of these changes is to add content for lifetime, thread-safety,
and additional concepts specific to the messaging clients.  The content
and format is intended to match the common blocks used by the HTTP-based
service clients as closely as reasonable.

(see: "Thread safety" and "Additional concepts" here: https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/storage/Azure.Storage.Blobs#thread-safety)
@jsquire jsquire merged commit 65b19a0 into Azure:master Feb 17, 2021
@jsquire jsquire deleted the messaging/readme-updates branch February 17, 2021 13:52
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. Docs Event Hubs Service Bus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants