From 897950e65842085371e2e9ca0f3a1e5ffee259ba Mon Sep 17 00:00:00 2001 From: Maximilian Grundke Date: Wed, 28 Feb 2018 10:36:12 +0100 Subject: [PATCH 1/2] Migrate in batches of 1000 records --- ...130172021_add_reached_full_score_to_request_for_comment.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/db/migrate/20180130172021_add_reached_full_score_to_request_for_comment.rb b/db/migrate/20180130172021_add_reached_full_score_to_request_for_comment.rb index ed1d27a9..c29919fd 100644 --- a/db/migrate/20180130172021_add_reached_full_score_to_request_for_comment.rb +++ b/db/migrate/20180130172021_add_reached_full_score_to_request_for_comment.rb @@ -1,8 +1,8 @@ class AddReachedFullScoreToRequestForComment < ActiveRecord::Migration def up add_column :request_for_comments, :full_score_reached, :boolean, default: false - RequestForComment.all.each { |rfc| - if (rfc.submission.present? && rfc.submission.exercise.has_user_solved(rfc.user)) + RequestForComment.find_each { |rfc| + if rfc.submission.present? and rfc.submission.exercise.has_user_solved(rfc.user) rfc.full_score_reached = true rfc.save end From 8d9a81410f9bccaeb735fd223a3717e140cd941c Mon Sep 17 00:00:00 2001 From: Maximilian Grundke Date: Wed, 28 Feb 2018 10:44:03 +0100 Subject: [PATCH 2/2] Make migration pseudo-reversible --- db/migrate/20180222145909_fix_timestamps_on_feedback.rb | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/db/migrate/20180222145909_fix_timestamps_on_feedback.rb b/db/migrate/20180222145909_fix_timestamps_on_feedback.rb index 95941c5a..f07372c7 100644 --- a/db/migrate/20180222145909_fix_timestamps_on_feedback.rb +++ b/db/migrate/20180222145909_fix_timestamps_on_feedback.rb @@ -1,6 +1,10 @@ class FixTimestampsOnFeedback < ActiveRecord::Migration - def change + def up change_column_default(:user_exercise_feedbacks, :created_at, nil) change_column_default(:user_exercise_feedbacks, :updated_at, nil) end + + def down + + end end