[Core] Don't check in a bunch of commented code into the main branch

[jzabroski]

API Change Needed

No

Background and motivation

I found a GitHub comment by Eric Sampson (Microsoft) suggesting we go look at how the Azure SDK implemented support for robust HttpClient usage. So, I checked out ServiceClient.cs and found over 40 lines of commented out code.

API / Feature Proposal

Can someone clean this up and address it?

https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/mgmtcommon/ClientRuntime/ClientRuntime/ServiceClient.cs

Time Constraints and Dependencies

The dependency would be that another Microsoft team claims this is a gold standard reference implementation, and it does not look like any production code I would use.

dotnet/aspnetcore#28385 (comment)

Stakeholders

@shahabhijeet

[christothes]

Hi @jzabroski - This may be what you were looking for https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/core/Azure.Core/src/Pipeline/HttpClientTransport.cs

[jzabroski]

@christothes Thank you, that is wonderful.