summaryrefslogtreecommitdiff
path: root/db/migrate/20191212140117_change_commit_user_mentions_commit_id_column_type.rb
blob: f205de3587e5b5615b17da2d6ce2eb884c8e88af (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
36
37
38
# frozen_string_literal: true

class ChangeCommitUserMentionsCommitIdColumnType < ActiveRecord::Migration[5.2]
  include Gitlab::Database::MigrationHelpers

  DOWNTIME = false

  disable_ddl_transaction!

  OLD_INDEX = 'commit_user_mentions_on_commit_id_and_note_id_index'
  OLD_TMP_INDEX = 'temp_commit_id_and_note_id_index'
  NEW_TMP_INDEX = 'temp_commit_id_for_type_change_and_note_id_index'
  NEW_INDEX = 'commit_id_and_note_id_index'

  # rubocop:disable Migration/PreventStrings
  def up
    # the initial index name is too long and fails during migration. Renaming the index first.
    add_concurrent_index :commit_user_mentions, [:commit_id, :note_id], name: OLD_TMP_INDEX
    remove_concurrent_index_by_name :commit_user_mentions, OLD_INDEX

    change_column_type_concurrently :commit_user_mentions, :commit_id, :string

    # change_column_type_concurrently creates a new index for new column `commit_id_for_type` based on existing
    # `temp_commit_id_and_note_id_index` naming it `temp_commit_id_for_type_change_and_note_id_index`, yet keeping
    # `temp_commit_id_and_note_id_index` for `commit_id`, that will be cleaned
    # by `cleanup_concurrent_column_type_change :commit_user_mentions, :commit_id` in a later migration.
    #
    # So we'll rename `temp_commit_id_for_type_change_and_note_id_index` to initialy intended name: `commit_id_and_note_id_index`.

    add_concurrent_index :commit_user_mentions, [:commit_id_for_type_change, :note_id], name: NEW_INDEX
    remove_concurrent_index_by_name :commit_user_mentions, NEW_TMP_INDEX
  end
  # rubocop:enable Migration/PreventStrings

  def down
    cleanup_concurrent_column_type_change :commit_user_mentions, :commit_id
  end
end