From 2017c5c62abde0d6f24e3afc120365ee1aaaca4b Mon Sep 17 00:00:00 2001 From: Sean McGivern Date: Thu, 16 Aug 2018 12:17:41 +0100 Subject: Fix routes N+1 in Issues::ReferencedMergeRequestsService#execute Sorting here needs the project routes to be loaded, including the namespace routes. --- lib/banzai/reference_parser/merge_request_parser.rb | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'lib/banzai') diff --git a/lib/banzai/reference_parser/merge_request_parser.rb b/lib/banzai/reference_parser/merge_request_parser.rb index a370ff5b5b3..9e5d55f72bc 100644 --- a/lib/banzai/reference_parser/merge_request_parser.rb +++ b/lib/banzai/reference_parser/merge_request_parser.rb @@ -14,11 +14,12 @@ module Banzai # Eager loading these ensures we don't end up running dozens of # queries in this process. target_project: [ - { namespace: :owner }, + { namespace: [:owner, :route] }, { group: [:owners, :group_members] }, :invited_groups, :project_members, - :project_feature + :project_feature, + :route ] }), self.class.data_attribute -- cgit v1.2.3