about summary refs log tree commit diff
path: root/db/migrate/20171122120436_add_index_account_and_reblog_of_id_to_statuses.rb
diff options
context:
space:
mode:
authorunarist <m.unarist@gmail.com>2017-11-28 04:22:27 +0900
committerEugen Rochko <eugen@zeonfederated.com>2017-11-27 20:22:27 +0100
commit7fb850e987c92a6b863cc1f66a6ec7318460f1de (patch)
tree95bea17ae5fba5b7b7071f510c5f60d353651efe /db/migrate/20171122120436_add_index_account_and_reblog_of_id_to_statuses.rb
parent1c5b0e333464d8da3de73e4886502c816cb8173e (diff)
Merge indexes for reblog on statuses table (#5831)
We added an index for `[account_id, reblog_of_id]`, but we already have a similar index for `reblog_of_id`. Those index will be bigger according to statuses count. For example, `reblog_of_id` index uses 800MB for 10GB statuses table.

So this patch swaps indexed columns like `[reblog_of_id, account_id]`, then it will covers both usage with single index.

Since those index creation may take a while, I've also disabled previous index creation.
Diffstat (limited to 'db/migrate/20171122120436_add_index_account_and_reblog_of_id_to_statuses.rb')
-rw-r--r--db/migrate/20171122120436_add_index_account_and_reblog_of_id_to_statuses.rb12
1 files changed, 9 insertions, 3 deletions
diff --git a/db/migrate/20171122120436_add_index_account_and_reblog_of_id_to_statuses.rb b/db/migrate/20171122120436_add_index_account_and_reblog_of_id_to_statuses.rb
index 49c1ce0a8..131e54b72 100644
--- a/db/migrate/20171122120436_add_index_account_and_reblog_of_id_to_statuses.rb
+++ b/db/migrate/20171122120436_add_index_account_and_reblog_of_id_to_statuses.rb
@@ -1,6 +1,12 @@
 class AddIndexAccountAndReblogOfIdToStatuses < ActiveRecord::Migration[5.1]
-  def change
-    commit_db_transaction
-    add_index :statuses, [:account_id, :reblog_of_id], algorithm: :concurrently
+  disable_ddl_transaction!
+
+  def up
+    # This index has been superseded by migration 20171125185353
+    # add_index :statuses, [:account_id, :reblog_of_id], algorithm: :concurrently
+  end
+
+  def down
+    remove_index :statuses, [:account_id, :reblog_of_id] if index_exists?(:statuses, [:account_id, :reblog_of_id])
   end
 end