From ca2100d012bd5028750c8161f3b6e1ac6cb822c3 Mon Sep 17 00:00:00 2001
From: Sam <sam.saffron@gmail.com>
Date: Wed, 10 Sep 2014 22:19:24 +1000
Subject: [PATCH] PERF: work around LIMIT breaking query plan

(CTE is a optimisation boundary, so limit no longer has any effect on plan)
---
 app/models/topic_tracking_state.rb | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/models/topic_tracking_state.rb b/app/models/topic_tracking_state.rb
index 2ed1bf5f0de..05334ccadae 100644
--- a/app/models/topic_tracking_state.rb
+++ b/app/models/topic_tracking_state.rb
@@ -130,6 +130,7 @@ class TopicTrackingState
     new = TopicQuery.new_filter(Topic, "xxx").where_values.join(" AND ").gsub!("'xxx'", treat_as_new_topic_clause)
 
     sql = <<SQL
+    WITH x AS (
     SELECT u.id AS user_id,
            topics.id AS topic_id,
            topics.created_at,
@@ -163,7 +164,8 @@ SQL
     if topic_id
       sql << " AND topics.id = :topic_id"
     end
-    sql << " ORDER BY topics.bumped_at DESC LIMIT 500"
+
+    sql << " ORDER BY topics.bumped_at DESC ) SELECT * FROM x LIMIT 500"
 
     SqlBuilder.new(sql)
       .map_exec(TopicTrackingState, user_ids: user_ids, topic_id: topic_id)