Merge | Port PR 499 to .NET Framework

[edwardneal]

This ports #499 from netcore to netfx.

Relates to #1261.
Relates to #2965.

Once this PR is finished, the next PR which touches SqlDataReader will be able to merge it.

I'd appreciate a CI run of this please.

[cheenamalhotra]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[codecov]

Codecov Report

Attention: Patch coverage is 88.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 66.30%. Comparing base (1247ca4) to head (b132943).
Report is 14 commits behind head on main.

Files with missing lines	Patch %	Lines
...etfx/src/Microsoft/Data/SqlClient/SqlDataReader.cs	88.00%	3 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (1247ca4) and HEAD (b132943). Click for more details.

HEAD has 1 upload less than BASE

Flag	BASE (1247ca4)	HEAD (b132943)
addons	1	0

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3255      +/-   ##
==========================================
- Coverage   72.69%   66.30%   -6.39%     
==========================================
  Files         303      292      -11     
  Lines       59718    59324     -394     
==========================================
- Hits        43414    39337    -4077     
- Misses      16304    19987    +3683     

Flag	Coverage Δ
addons	?
netcore	69.02% <ø> (-6.09%)	⬇️
netfx	65.23% <88.00%> (-6.15%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[mdaigle]

Looks like basically a 1:1 port and tests pass ✔️