Skip to content

SQLite is bugged with distributed transactions: disable distributed tests #1506

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

Merged

Conversation

fredericDelaporte
Copy link
Member

@fredericDelaporte fredericDelaporte commented Dec 27, 2017

SQLite does not actually implement two phases commit and has threading issues when participating into a distributed transaction.
https://system.data.sqlite.org/index.html/tktview/5cee5409f84da5f62172

An example of such threading issue is available here. This is a build from the period of time where the TeamCity build agent was dual cored. It seems the trouble does not occur (or way less) on single core instances.

…ests.

SQLite does not actually implement two phases commit and has threading issues when participating into a distributed transaction.
https://system.data.sqlite.org/index.html/tktview/5cee5409f84da5f62172
@fredericDelaporte fredericDelaporte merged commit 4b8eab4 into nhibernate:master Dec 28, 2017
@fredericDelaporte fredericDelaporte deleted the DistributedSQLite branch December 28, 2017 09:53
@fredericDelaporte fredericDelaporte added this to the 5.1 milestone Dec 28, 2017
@fredericDelaporte fredericDelaporte changed the title SQLite is bugged with distributed transactions: disable distributed t… SQLite is bugged with distributed transactions: disable distributed tests. Mar 12, 2018
@fredericDelaporte fredericDelaporte changed the title SQLite is bugged with distributed transactions: disable distributed tests. SQLite is bugged with distributed transactions: disable distributed tests Mar 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants