Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Remove unused indices #782

Merged
merged 3 commits into from
May 13, 2016
Merged

Remove unused indices #782

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
5e50058
Remove unused indices
erikjohnston May 13, 2016
9295fa3
Annotate the removed indicies with why they were removed.
May 13, 2016
c9aff07
Remove topics table
erikjohnston May 13, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 15 additions & 18 deletions synapse/storage/schema/delta/32/remove_indices.sql
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,27 +16,24 @@

-- The following indices are redundant, other indices are equivalent or
-- supersets
DROP INDEX IF EXISTS events_room_id;
DROP INDEX IF EXISTS events_order;
DROP INDEX IF EXISTS events_topological_ordering;
DROP INDEX IF EXISTS events_stream_ordering;
DROP INDEX IF EXISTS state_groups_id;
DROP INDEX IF EXISTS event_to_state_groups_id;
DROP INDEX IF EXISTS event_push_actions_room_id_event_id_user_id_profile_tag;
DROP INDEX IF EXISTS event_push_actions_room_id_user_id;

DROP INDEX IF EXISTS event_destinations_id;
DROP INDEX IF EXISTS st_extrem_id;
DROP INDEX IF EXISTS event_content_hashes_id;
DROP INDEX IF EXISTS event_signatures_id;
DROP INDEX IF EXISTS event_edge_hashes_id;
DROP INDEX IF EXISTS redactions_event_id;
DROP INDEX IF EXISTS remote_media_cache_thumbnails_media_id;
DROP INDEX IF EXISTS room_hosts_room_id;
DROP INDEX IF EXISTS event_search_ev_ridx;
DROP INDEX IF EXISTS events_room_id; -- Prefix of events_room_stream
DROP INDEX IF EXISTS events_order; -- Prefix of events_order_topo_stream_room
DROP INDEX IF EXISTS events_topological_ordering; -- Prefix of events_order_topo_stream_room
DROP INDEX IF EXISTS events_stream_ordering; -- Duplicate of PRIMARY KEY
DROP INDEX IF EXISTS state_groups_id; -- Duplicate of PRIMARY KEY
DROP INDEX IF EXISTS event_to_state_groups_id; -- Duplicate of PRIMARY KEY
DROP INDEX IF EXISTS event_push_actions_room_id_event_id_user_id_profile_tag; -- Duplicate of UNIQUE CONSTRAINT

DROP INDEX IF EXISTS event_destinations_id; -- Prefix of UNIQUE CONSTRAINT
DROP INDEX IF EXISTS st_extrem_id; -- Prefix of UNIQUE CONSTRAINT
DROP INDEX IF EXISTS event_content_hashes_id; -- Prefix of UNIQUE CONSTRAINT
DROP INDEX IF EXISTS event_signatures_id; -- Prefix of UNIQUE CONSTRAINT
DROP INDEX IF EXISTS event_edge_hashes_id; -- Prefix of UNIQUE CONSTRAINT
DROP INDEX IF EXISTS redactions_event_id; -- Duplicate of UNIQUE CONSTRAINT
DROP INDEX IF EXISTS room_hosts_room_id; -- Prefix of UNIQUE CONSTRAINT

-- The following indices were unused
DROP INDEX IF EXISTS remote_media_cache_thumbnails_media_id;
DROP INDEX IF EXISTS evauth_edges_auth_id;
DROP INDEX IF EXISTS topics_room_id;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that either I'd want to keep this index or get rid of the entire table given that it is write only in synapse, and can be fairly easily constructed from the state tables for ad-hoc queries.

DROP INDEX IF EXISTS presence_stream_state;