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

Verify that distributed transactions are working on .NET 7 #1145

Closed
bording opened this issue Jan 20, 2023 · 8 comments
Closed

Verify that distributed transactions are working on .NET 7 #1145

bording opened this issue Jan 20, 2023 · 8 comments

Comments

@bording
Copy link
Member

bording commented Jan 20, 2023

.NET 7 has added support for distributed transactions when running on Windows.

We should enable our TransactionScope tests for .NET 7 / Windows to verify that everything is working correctly.

Once that is done, we should update our transaction docs to reflect these changes.

@andreasohlund
Copy link
Member

@bording #1243 proves that they work, lets close this one?

@bording
Copy link
Member Author

bording commented Sep 15, 2023

Might be a good idea to leave this open until documentation is changed? We could also update the docs to explain how it would be possible to use with the existing packages by adding TransactionManager.ImplicitDistributedTransactions = true in a project.

@andreasohlund
Copy link
Member

Turns out that a bug that we found and MS fixed in the .netframework driver didn't get ported to the .net version of the driver and is now blocking us from supporting transaction scopes with DTC. Note that transaction scopes do work and its just when escalating to DTC the mentioned deadlock happens.

@danielmarbach has submitted a PR to fix this and once merged and released we can proceed with this

@SimonCropp
Copy link
Contributor

there is a preview nuget with @danielmarbach fix if u want to test it

@SimonCropp
Copy link
Contributor

i tested in Nsb.Attachments and it worked

@bording
Copy link
Member Author

bording commented Dec 14, 2023

DTC tests are passing in #1267

@danielmarbach
Copy link
Contributor

5.1.4 of the MS data client contains the fix that I contributed

@andreasohlund
Copy link
Member

It's been applied, should we close this one?

@bording bording closed this as completed Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants