PERF: Use different column for better query plan (#14748)

Using topics.id provides a better query plan than posts.topic_id which speeds up search by almost 50%.
discourse · Oct 28, 2021 · ba3078e · ba3078e
1 parent a2851b5
commit ba3078e
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/app/models/post.rb b/app/models/post.rb
@@ -86,8 +86,8 @@ class Post < ActiveRecord::Base
 
   scope :private_posts_for_user, ->(user) do
     where(
-      "posts.topic_id IN (#{Topic::PRIVATE_MESSAGES_SQL_USER})
-      OR posts.topic_id IN (#{Topic::PRIVATE_MESSAGES_SQL_GROUP})",
+      "topics.id IN (#{Topic::PRIVATE_MESSAGES_SQL_USER})
+      OR topics.id IN (#{Topic::PRIVATE_MESSAGES_SQL_GROUP})",
       user_id: user.id
     )
   end