20171114160904_merge_requests_updated_by_id_foreign_key.rb 1.18 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 MergeRequestsUpdatedByIdForeignKey < 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 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46
  include Gitlab::Database::MigrationHelpers

  # Set this constant to true if this migration requires downtime.
  DOWNTIME = false

  disable_ddl_transaction!

  class MergeRequest < ActiveRecord::Base
    include EachBatch

    self.table_name = 'merge_requests'

    def self.with_orphaned_updaters
      where('NOT EXISTS (SELECT true FROM users WHERE merge_requests.updated_by_id = users.id)')
        .where('updated_by_id IS NOT NULL')
    end
  end

  def up
    MergeRequest.with_orphaned_updaters.each_batch(of: 100) do |batch|
      batch.update_all(updated_by_id: nil)
    end

    add_concurrent_index(
      :merge_requests,
      :updated_by_id,
      where: 'updated_by_id IS NOT NULL'
    )

    add_concurrent_foreign_key(
      :merge_requests,
      :users,
      column: :updated_by_id,
      on_delete: :nullify
    )
  end

  def down
    remove_foreign_key_without_error(:merge_requests, column: :updated_by_id)
    remove_concurrent_index(:merge_requests, :updated_by_id)
  end
end