Schedule background migration only when it is needed

5292eb65 · Grzegorz Bizon · 02bb40e2 · 5292eb65
Commit 5292eb65 authored Jun 28, 2017 by Grzegorz Bizon
Show whitespace changes
Inline Side-by-side

Showing with 6 additions and 4 deletions

db/post_migrate/20170628080858_migrate_stage_id_reference_in_background.rb ...0170628080858_migrate_stage_id_reference_in_background.rb +6 -4

No files found.
--- a/db/post_migrate/20170628080858_migrate_stage_id_reference_in_background.rb
+++ b/db/post_migrate/20170628080858_migrate_stage_id_reference_in_background.rb
@@ -12,7 +12,9 @@ class MigrateStageIdReferenceInBackground < ActiveRecord::Migration
  end

  def up
-    Build.find_in_batches(batch_size: BATCH_SIZE).with_index do |builds, batch|
+    Build.where(stage_id: nil)
+      .find_in_batches(batch_size: BATCH_SIZE)
+      .with_index do |builds, batch|
        migrations = builds.map { |build| [MIGRATION, [build.id]] }

        BackgroundMigrationWorker.perform_bulk(*migrations)