diff options
| author | Erik Johnston <erik@matrix.org> | 2021-02-24 10:13:53 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-02-24 10:13:53 +0000 |
| commit | 0b5c967813a8e5f3338b6ecd5f3742e91b8a100b (patch) | |
| tree | 8001d34fb268c378624382d13a94c6e2c0b047e4 /synapse/storage/databases/state | |
| parent | Add a comment about systemd-python. (#9464) (diff) | |
| download | synapse-0b5c967813a8e5f3338b6ecd5f3742e91b8a100b.tar.xz | |
Refactor to ensure we call check_consistency (#9470)
The idea here is to stop people forgetting to call `check_consistency`. Folks can still just pass in `None` to the new args in `build_sequence_generator`, but hopefully they won't.
Diffstat (limited to 'synapse/storage/databases/state')
| -rw-r--r-- | synapse/storage/databases/state/store.py | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/synapse/storage/databases/state/store.py b/synapse/storage/databases/state/store.py index b16b9905d8..e2240703a7 100644 --- a/synapse/storage/databases/state/store.py +++ b/synapse/storage/databases/state/store.py @@ -97,10 +97,12 @@ class StateGroupDataStore(StateBackgroundUpdateStore, SQLBaseStore): return txn.fetchone()[0] self._state_group_seq_gen = build_sequence_generator( - self.database_engine, get_max_state_group_txn, "state_group_id_seq" - ) - self._state_group_seq_gen.check_consistency( - db_conn, table="state_groups", id_column="id" + db_conn, + self.database_engine, + get_max_state_group_txn, + "state_group_id_seq", + table="state_groups", + id_column="id", ) @cached(max_entries=10000, iterable=True) |