From 43f0884660f1df1f6c04bd4d29bc9d76bf0534f5 Mon Sep 17 00:00:00 2001 From: Vinoth Kannan Date: Fri, 23 Feb 2018 20:05:51 +0530 Subject: [PATCH] PERF: Remove N+1 queries on topic list page. --- app/serializers/concerns/topic_tags_mixin.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/serializers/concerns/topic_tags_mixin.rb b/app/serializers/concerns/topic_tags_mixin.rb index 2fdbc3deb6c..6ad88b47ea2 100644 --- a/app/serializers/concerns/topic_tags_mixin.rb +++ b/app/serializers/concerns/topic_tags_mixin.rb @@ -8,7 +8,8 @@ module TopicTagsMixin end def tags - topic.tags.pluck(:name) + # Calling method `pluck` along with `includes` causing N+1 queries + topic.tags.map(&:name) end def topic