Azure / azure-sdk-for-net

This repository is for active development of the Azure SDK for .NET. For consumers of the SDK we recommend visiting our public developer docs at https://learn.microsoft.com/dotnet/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-net.
MIT License
5.47k stars 4.8k forks source link

Update remarks for StorageExtensions.CreateServiceTimeoutScope(Nullable<TimeSpan>) method to clarify SDK vs service behavior #32973

Open schoag-msft opened 1 year ago

schoag-msft commented 1 year ago

The current statement in the remarks:

"Consider passing a CancellationToken to client methods and properly sizing NetworkTimeout when configuring storage clients as prefered way of enforcing upper boundary of execution time."

should be updated as it is referring to the operation time of the application method end to end, since the server timeout does not account for network travel time. We should clarify when this method should be used vs a servertimeout.


Document Details

Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

jsquire commented 1 year ago

//cc: @amnguye, @seanmcc-msft