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

Add Support for MultiSubnetFailover when using a High Availability Group #337

Open
pip8786 opened this issue Apr 2, 2024 · 6 comments
Open

Comments

@pip8786
Copy link

pip8786 commented Apr 2, 2024

Currently Prisma doesn't support the MultiSubnetFailover option of SQL Server. See prisma/prisma#9187. I think this would require support at the driver level so I'm adding an issue here.

Other info: https://learn.microsoft.com/en-us/dotnet/framework/data/adonet/sql/sqlclient-support-for-high-availability-disaster-recovery

@dariusj18
Copy link

@pip8786
Copy link
Author

pip8786 commented Apr 18, 2024

Thanks @dariusj18 . That seems relatively simple so maybe the maintainers could implement this pretty quickly in their rust code. I tried looking at the connection code and I'm definitely not qualified to try a PR in Rust.

@dariusj18
Copy link

I'd give it stab, but I am kinda worried that PRs won't be merged in. There hasn't been much activity and a few people that have commented say things like "I'm not the owner of this crate anymore"

So I don't know if anyone is really maintaining it.

@dariusj18
Copy link

Upon further further evaluation, the TcpStream connection is created in a few places

async fn connect_named(builder: &crate::client::Config) -> crate::Result<Self> {

async fn connect_named(builder: &Config) -> crate::Result<Self> {

async fn connect_named(builder: &Config) -> crate::Result<Self> {

it seems that these exist for different library compatibilities.

@pip8786
Copy link
Author

pip8786 commented Apr 26, 2024

So do you think we need to post an issue on prisma-engines as well or is this still the right place?

@dariusj18
Copy link

This is the right place.

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

No branches or pull requests

2 participants