Andrew Harper

Yeah good question how can we force this behavior? What about something like this: 1. create a connection 2. run something like `kill @@spid` (not sure if this can be...

> Just came across this from the rails sql server adapter source. Is there a blocker to merging this? cc @metaskills There's no real hard blockers, however it would be...

@wpolicarpo @bvogelzang was able to force various timeouts in https://github.com/rails-sqlserver/tiny_tds/pull/481, so maybe something can be done here that way? Alternately if there isn't enough energy to add a test, I...

> Do you have a rough idea of when that PR will be merged? I expect we can merge it when the CI tests pass. I don't recall off the...

> I expect we can merge it when the CI tests pass. I don't recall off the top of my head what the blocker there was @bvogelzang got the tests...

@wpolicarpo I saw you added a testing commit here https://github.com/rails-sqlserver/tiny_tds/pull/486/commits/1c2f56e5ff740c7a745bf566c455bacd6d1fbf1b, what do you think about adding it to this branch? Did Toxiproxy have the intended effect?

@wpolicarpo I moved your commit that added a test from https://github.com/rails-sqlserver/tiny_tds/pull/486 to here and rebased on latest master (I realize this may be slightly disruptive since I force pushed, but...

> Is there any problems to prevent from this PR get progress? The "problem" is that I still need to get a test lab set up to confirm this actually...

Alright, I had some time to revisit this branch. I have an AG lab set up locally to verify that setting the new option does indeed work as expected. There's...

This should maybe change from a bool to a symbol enumeration, to support Application Intent Read Write https://docs.microsoft.com/en-us/sql/database-engine/availability-groups/windows/secondary-replica-connection-redirection-always-on-availability-groups?view=sql-server-ver15