From cac7ea35cfb91003e4bd95106ed29963d34b6f8f Mon Sep 17 00:00:00 2001 From: David Taylor Date: Tue, 17 Nov 2020 12:07:16 +0000 Subject: [PATCH] PERF: Simplify topic serialization for user summary page (#11236) ListableTopicSerializer includes many attributes which we are not using, and is likely to cause N+1s when not used in conjunction with TopicQuery. Using the BasicTopicSerializer means that no other tables are required. --- app/serializers/user_summary_serializer.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/serializers/user_summary_serializer.rb b/app/serializers/user_summary_serializer.rb index 3a7c9d8fb34..65d0250de13 100644 --- a/app/serializers/user_summary_serializer.rb +++ b/app/serializers/user_summary_serializer.rb @@ -2,8 +2,8 @@ class UserSummarySerializer < ApplicationSerializer - class TopicSerializer < ListableTopicSerializer - attributes :category_id, :like_count + class TopicSerializer < BasicTopicSerializer + attributes :category_id, :like_count, :created_at end class ReplySerializer < ApplicationSerializer