Different behavior for Routing in netcore and netfx

[JRahnama]

There are two different behavior for Routing in SqlClient

in .Net inside SqlInternalConnectionTds we have

if (routingAttempts > 0)
                        {
                            throw SQL.ROR_RecursiveRoutingNotSupported(this);
                        }

Where as the same place in .NetFramework we have

if (routingAttempts > _maxNumberOfRedirectRoute)
                        {
                            throw SQL.ROR_RecursiveRoutingNotSupported(this);
                        }

_maxNumberOfRedirectRout is 10

[JRahnama]

@David-Engel, @saurabh500 can you confirm that netcore should follow the same logic as netfx?

[David-Engel]

@JRahnama

I noticed this a little while ago and looked into the history a bit. The max routing attempts changes was included as part of the Azure read only routing support changes in SDS netfx. I saved a branch of what I thought were the relevant changes required from bring everything from the netfx to the netcore MDS code: https://github.com/dotnet/SqlClient/compare/main...David-Engel:SqlClient:port?expand=1

[JRahnama]

@JRahnama

I noticed this a little while ago and looked into the history a bit. The max routing attempts changes was included as part of the Azure read only routing support changes in SDS netfx. I saved a branch of what I thought were the relevant changes required from bring everything from the netfx to the netcore MDS code: https://github.com/dotnet/SqlClient/compare/main...David-Engel:SqlClient:port?expand=1

Thanks David, I have updated the PR #2873 with some extra changes and addressed some issues in your changes.