summaryrefslogtreecommitdiff
path: root/db/migrate/20170724214302_add_lower_path_index_to_redirect_routes.rb
blob: a770ff63b4e1ee5fbe9c8a900254d9387c7469ac (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
# See http://doc.gitlab.com/ce/development/migration_style_guide.html
# for more information on how to write migrations for GitLab.

class AddLowerPathIndexToRedirectRoutes < ActiveRecord::Migration
  include Gitlab::Database::MigrationHelpers

  DOWNTIME = false
  INDEX_NAME = 'index_on_redirect_routes_lower_path'

  disable_ddl_transaction!

  def up
    return unless Gitlab::Database.postgresql?

    execute "CREATE INDEX CONCURRENTLY #{INDEX_NAME} ON redirect_routes (LOWER(path));"
  end

  def down
    return unless Gitlab::Database.postgresql?

    # Why not use remove_concurrent_index_by_name?
    #
    # `index_exists?` doesn't work on this index. Perhaps this is related to the
    # fact that the index doesn't show up in the schema. And apparently it isn't
    # trivial to write a query that checks for an index. BUT there is a
    # convenient `IF EXISTS` parameter for `DROP INDEX`.
    if supports_drop_index_concurrently?
      disable_statement_timeout do
        execute "DROP INDEX CONCURRENTLY IF EXISTS #{INDEX_NAME};"
      end
    else
      execute "DROP INDEX IF EXISTS #{INDEX_NAME};"
    end
  end
end