summaryrefslogtreecommitdiff
path: root/db/migrate/20171106150657_issues_updated_by_id_foreign_key.rb
blob: b2992b1ff5dc647f5189e2c45068b36ed562f8ad (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
39
40
41
42
43
44
45
# See http://doc.gitlab.com/ce/development/migration_style_guide.html
# for more information on how to write migrations for GitLab.

class IssuesUpdatedByIdForeignKey < ActiveRecord::Migration[4.2]
  include Gitlab::Database::MigrationHelpers

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

  disable_ddl_transaction!

  class Issue < ActiveRecord::Base
    include EachBatch

    self.table_name = 'issues'

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

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

    # This index is only used for foreign keys, and those in turn will always
    # specify a value. As such we can add a WHERE condition to make the index
    # smaller.
    add_concurrent_index(:issues, :updated_by_id, where: 'updated_by_id IS NOT NULL')

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

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