20170724214302_add_lower_path_index_to_redirect_routes.rb 1.12 KB
Newer Older
1 2 3
# See http://doc.gitlab.com/ce/development/migration_style_guide.html
# for more information on how to write migrations for GitLab.

4
class AddLowerPathIndexToRedirectRoutes < ActiveRecord::Migration[4.2]
5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27
  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?
28
      disable_statement_timeout do
29 30
        execute "DROP INDEX CONCURRENTLY IF EXISTS #{INDEX_NAME};"
      end
31 32 33 34 35
    else
      execute "DROP INDEX IF EXISTS #{INDEX_NAME};"
    end
  end
end