Move temp table creation into the prepare job

* Hopefully fixes spec failures in which the table doesn’t exist
* Decouples the background migration from the post-deploy migration, e.g. we could easily run it again even though the table is dropped when finished.

1 files changed, 0 insertions, 20 deletions

diff --git a/db/post_migrate/20171103140253_track_untracked_uploads.rb b/db/post_migrate/20171103140253_track_untracked_uploads.rb
index 7a34abc85ee..548a94d2d38 100644
--- a/db/post_migrate/20171103140253_track_untracked_uploads.rb
+++ b/db/post_migrate/20171103140253_track_untracked_uploads.rb
@@ -10,8 +10,6 @@ class TrackUntrackedUploads < ActiveRecord::Migration
   MIGRATION = 'PrepareUntrackedUploads'
 
   def up
-    ensure_temporary_tracking_table_exists
-
     BackgroundMigrationWorker.perform_async(MIGRATION)
   end
 
@@ -20,22 +18,4 @@ class TrackUntrackedUploads < ActiveRecord::Migration
       drop_table :untracked_files_for_uploads
     end
   end
-
-  def ensure_temporary_tracking_table_exists
-    unless table_exists?(:untracked_files_for_uploads)
-      create_table :untracked_files_for_uploads do |t|
-        t.string :path, limit: 600, null: false
-        t.boolean :tracked, default: false, null: false
-        t.timestamps_with_timezone null: false
-      end
-    end
-
-    unless index_exists?(:untracked_files_for_uploads, :path)
-      add_index :untracked_files_for_uploads, :path, unique: true
-    end
-
-    unless index_exists?(:untracked_files_for_uploads, :tracked)
-      add_index :untracked_files_for_uploads, :tracked
-    end
-  end
 end