Commit e53169c9 authored by Douwe Maan's avatar Douwe Maan

Resolve N+1 query issue with discussions

parent 1ac12698
...@@ -43,7 +43,12 @@ module Noteable ...@@ -43,7 +43,12 @@ module Noteable
end end
def resolvable_discussions def resolvable_discussions
@resolvable_discussions ||= discussion_notes.resolvable.discussions(self) @resolvable_discussions ||=
if defined?(@discussions)
@discussions.select(&:resolvable?)
else
discussion_notes.resolvable.discussions(self)
end
end end
def discussions_resolvable? def discussions_resolvable?
......
...@@ -21,7 +21,8 @@ class Discussion ...@@ -21,7 +21,8 @@ class Discussion
end end
def self.build_collection(notes, context_noteable = nil) def self.build_collection(notes, context_noteable = nil)
notes.group_by { |n| n.discussion_id(context_noteable) }.values.map { |notes| build(notes, context_noteable) } grouped_notes = notes.includes(:noteable).group_by { |n| n.discussion_id(context_noteable) }
grouped_notes.values.map { |notes| build(notes, context_noteable) }
end end
# Returns an alphanumeric discussion ID based on `build_discussion_id` # Returns an alphanumeric discussion ID based on `build_discussion_id`
......
---
title: Resolve N+1 query issue with discussions
merge_request:
author:
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment