Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Avoid sql connection timeout #1414

Merged
merged 3 commits into from Jun 25, 2021
Merged

Avoid sql connection timeout #1414

merged 3 commits into from Jun 25, 2021

Conversation

MGibson1
Copy link
Member

Overview

We finally pegged down the Sql connection closed issue to being related to this issue: dotnet/SqlClient#54.

The Issue provides a workaround: user MARS. We can mitigate by altering the connection string for the methods which have trouble.

Data Table creation is slow. connection may be timing out while we create it.
@MGibson1 MGibson1 requested review from cscharf and a team June 25, 2021 18:27
@MGibson1 MGibson1 merged commit 9841502 into master Jun 25, 2021
@MGibson1 MGibson1 deleted the avoid-sql-connection-timeout branch June 25, 2021 18:37
MGibson1 added a commit that referenced this pull request Jun 25, 2021
* Creat TVP prior to opening sql connection

Data Table creation is slow. connection may be timing out while we create it.

* USe MARS-enabled connections to fix connection issue

dotnet/SqlClient#54
(cherry picked from commit 9841502)
@MGibson1
Copy link
Member Author

cherry picked to rc

dlundgren pushed a commit to dlundgren/server that referenced this pull request Dec 14, 2021
* Creat TVP prior to opening sql connection

Data Table creation is slow. connection may be timing out while we create it.

* USe MARS-enabled connections to fix connection issue

dotnet/SqlClient#54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants