diff options
author | Erik Johnston <erik@matrix.org> | 2020-10-07 15:15:57 +0100 |
---|---|---|
committer | Erik Johnston <erik@matrix.org> | 2020-10-07 17:08:58 +0100 |
commit | fa8934b175467d589dd34fae18639cac0d738fc9 (patch) | |
tree | b47f6e6628a61a5b6689ed576c98ec59b3542a8d /tests | |
parent | Add Ubuntu 20.10 (Groovy Gorilla) to build scripts. (#8475) (diff) | |
download | synapse-fa8934b175467d589dd34fae18639cac0d738fc9.tar.xz |
Reduce serialization errors in MultiWriterIdGen (#8456)
We call `_update_stream_positions_table_txn` a lot, which is an UPSERT that can conflict in `REPEATABLE READ` isolation level. Instead of doing a transaction consisting of a single query we may as well run it outside of a transaction.
Diffstat (limited to 'tests')
-rw-r--r-- | tests/storage/test_base.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/storage/test_base.py b/tests/storage/test_base.py index 40ba652248..eac7e4dcd2 100644 --- a/tests/storage/test_base.py +++ b/tests/storage/test_base.py @@ -56,6 +56,7 @@ class SQLBaseStoreTestCase(unittest.TestCase): engine = create_engine(sqlite_config) fake_engine = Mock(wraps=engine) fake_engine.can_native_upsert = False + fake_engine.in_transaction.return_value = False db = DatabasePool(Mock(), Mock(config=sqlite_config), fake_engine) db._db_pool = self.db_pool |