summary refs log tree commit diff
path: root/synapse/storage/event_push_actions.py
diff options
context:
space:
mode:
authorDavid Baker <dave@matrix.org>2016-09-09 18:54:54 +0100
committerDavid Baker <dave@matrix.org>2016-09-09 18:54:54 +0100
commitf2acc3dcf9cf213948ce3d2ebf12c3202abe97fd (patch)
treed6e28138a42a14a76ce4abc250dc0b1c3362a0c2 /synapse/storage/event_push_actions.py
parentImplement `only=highlight` on `/notifications` (diff)
downloadsynapse-f2acc3dcf9cf213948ce3d2ebf12c3202abe97fd.tar.xz
Add index to event_push_actions
and remove room_id caluse so it uses it

Mostly from @negativemjark
Diffstat (limited to 'synapse/storage/event_push_actions.py')
-rw-r--r--synapse/storage/event_push_actions.py4
1 files changed, 3 insertions, 1 deletions
diff --git a/synapse/storage/event_push_actions.py b/synapse/storage/event_push_actions.py
index dedf517cfa..a67c886f9a 100644
--- a/synapse/storage/event_push_actions.py
+++ b/synapse/storage/event_push_actions.py
@@ -353,12 +353,14 @@ class EventPushActionsStore(SQLBaseStore):
                     before_clause += " "
                 before_clause += "AND epa.highlight = 1"
 
+            # NB. This assumes event_ids are globally unique since
+            # it makes the query easier to index
             sql = (
                 "SELECT epa.event_id, epa.room_id,"
                 " epa.stream_ordering, epa.topological_ordering,"
                 " epa.actions, epa.profile_tag, e.received_ts"
                 " FROM event_push_actions epa, events e"
-                " WHERE epa.room_id = e.room_id AND epa.event_id = e.event_id"
+                " WHERE epa.event_id = e.event_id"
                 " AND epa.user_id = ? %s"
                 " ORDER BY epa.stream_ordering DESC"
                 " LIMIT ?"