diff options
author | Erik Johnston <erik@matrix.org> | 2016-09-12 16:55:01 +0100 |
---|---|---|
committer | Erik Johnston <erik@matrix.org> | 2016-09-12 16:55:01 +0100 |
commit | c94de0ab606e5bbe7331a58456204a5775df779b (patch) | |
tree | 18b018ed1b42dc9dc8108d839ed659514775be80 /synapse/storage/background_updates.py | |
parent | Merge pull request #1107 from matrix-org/erikj/backfill_none (diff) | |
download | synapse-c94de0ab606e5bbe7331a58456204a5775df779b.tar.xz |
Add WHERE clause support to index creation
Diffstat (limited to 'synapse/storage/background_updates.py')
-rw-r--r-- | synapse/storage/background_updates.py | 24 |
1 files changed, 15 insertions, 9 deletions
diff --git a/synapse/storage/background_updates.py b/synapse/storage/background_updates.py index 003f5ba203..94b2bcc54a 100644 --- a/synapse/storage/background_updates.py +++ b/synapse/storage/background_updates.py @@ -219,7 +219,7 @@ class BackgroundUpdateStore(SQLBaseStore): self._background_update_handlers[update_name] = update_handler def register_background_index_update(self, update_name, index_name, - table, columns): + table, columns, where_clause=None): """Helper for store classes to do a background index addition To use: @@ -243,14 +243,20 @@ class BackgroundUpdateStore(SQLBaseStore): conc = True else: conc = False - - sql = "CREATE INDEX %(conc)s %(name)s ON %(table)s (%(columns)s)" \ - % { - "conc": "CONCURRENTLY" if conc else "", - "name": index_name, - "table": table, - "columns": ", ".join(columns), - } + # We don't use partial indices on SQLite as it wasn't introduced + # until 3.8, and wheezy has 3.7 + where_clause = None + + sql = ( + "CREATE INDEX %(conc)s %(name)s ON %(table)s (%(columns)s)" + " %(where_clause)s" + ) % { + "conc": "CONCURRENTLY" if conc else "", + "name": index_name, + "table": table, + "columns": ", ".join(columns), + "where_clause": "WHERE " + where_clause if where_clause else "" + } def create_index_concurrently(conn): conn.rollback() |