From 95d13bf68ad51295a1c6bb5738e81715a80494b7 Mon Sep 17 00:00:00 2001
From: Sam <sam.saffron@gmail.com>
Date: Fri, 25 Jul 2014 13:21:10 +1000
Subject: [PATCH] PERF: move avg time calculation to daily
---
app/jobs/scheduled/calculate_avg_time.rb | 15 +++++++++++++++
app/jobs/scheduled/periodical_updates.rb | 3 ---
2 files changed, 15 insertions(+), 3 deletions(-)
create mode 100644 app/jobs/scheduled/calculate_avg_time.rb
diff --git a/app/jobs/scheduled/calculate_avg_time.rb b/app/jobs/scheduled/calculate_avg_time.rb
new file mode 100644
index 000000000..e842285ab
--- /dev/null
+++ b/app/jobs/scheduled/calculate_avg_time.rb
@@ -0,0 +1,15 @@
+module Jobs
+
+ class CalculateAvgTime < Jobs::Scheduled
+ every 1.day
+
+ # PERF: these calculations can become exceedingly expnsive
+ # they run a huge gemoetric mean and are hard to optimise
+ # defer to only run once a day
+ def execute(args)
+ # Update the average times
+ Post.calculate_avg_time(2.days.ago)
+ Topic.calculate_avg_time(2.days.ago)
+ end
+ end
+end
diff --git a/app/jobs/scheduled/periodical_updates.rb b/app/jobs/scheduled/periodical_updates.rb
index 27aaac4d4..2266ca17f 100644
--- a/app/jobs/scheduled/periodical_updates.rb
+++ b/app/jobs/scheduled/periodical_updates.rb
@@ -8,9 +8,6 @@ module Jobs
every 15.minutes
def execute(args)
- # Update the average times
- Post.calculate_avg_time(1.day.ago)
- Topic.calculate_avg_time(1.day.ago)
# Feature topics in categories
CategoryFeaturedTopic.feature_topics