Don't hammer the database on score calculation

whatupdave · whatupdave · commit 43323c50979c · 2017-05-29T09:50:59.000-07:00
diff --git a/app/models/article.rb b/app/models/article.rb
@@ -91,7 +91,7 @@ def cacluate_content_quality_score
     factor * (weight = 20)
   end
 
-  def cacluate_score
+  def calculate_score
     return 0 if bad_content?
     half_life      = 2.days.to_i
     # gravity        = 1.8 #used to look at upvote_velocity(1.week.ago)
@@ -124,7 +124,7 @@ def public_id_blank?
   end
 
   def cache_calculated_score!
-    self.score = cacluate_score
+    self.score = calculate_score
   end
 
   def display_tags
diff --git a/lib/tasks/cache.rake b/lib/tasks/cache.rake
@@ -7,9 +7,10 @@ namespace :cache do
   namespace :score do
     task :recalculate => :environment do
       ActiveRecord::Base.logger.level = Logger::INFO #hide sql output
-      Protip.order(created_at: :asc).find_each do |p|
-        score = p.cacluate_score
+      Protip.order(created_at: :asc).find_each(batch_size: 100) do |p|
+        score = p.calculate_score
         p.update_column(:score, score)
+        sleep 0.1
       end
     end
   end
