Commit 5d488073 authored by Dmitry Gruzd's avatar Dmitry Gruzd

Fix N+1 queries for milestone search

parent aed677b9
......@@ -89,6 +89,10 @@ class Milestone < ApplicationRecord
.order(:project_id, :group_id, :due_date).select('DISTINCT ON (project_id, group_id) id')
end
def self.with_web_entity_associations
preload(:group, project: [:project_feature, group: [:parent], namespace: :route])
end
def participants
User.joins(assigned_issues: :milestone).where("milestones.id = ?", id).distinct
end
......
......@@ -10,7 +10,8 @@ class SearchServicePresenter < Gitlab::View::Presenter::Delegated
issues: :with_web_entity_associations,
merge_requests: :with_web_entity_associations,
epics: :with_web_entity_associations,
notes: :with_web_entity_associations
notes: :with_web_entity_associations,
milestones: :with_web_entity_associations
}.freeze
SORT_ENABLED_SCOPES = %w(issues merge_requests).freeze
......
---
title: Fix N+1 for searching milestone scope
merge_request: 57715
author:
type: performance
......@@ -10,6 +10,15 @@ module EE
has_many :boards
end
class_methods do
extend ::Gitlab::Utils::Override
override :with_web_entity_associations
def with_web_entity_associations
super.preload(project: [group: [:saml_provider]])
end
end
def supports_milestone_charts?
resource_parent&.feature_available?(:milestone_charts) && weight_available?
end
......
......@@ -78,9 +78,7 @@ RSpec.describe 'Global elastic search', :elastic, :sidekiq_inline do
let(:object) { :milestone }
let(:creation_args) { { project: project } }
let(:path) { search_path(search: '*', scope: 'milestones') }
# N+1 queries still exist and will be fixed per
# https://gitlab.com/gitlab-org/gitlab/-/issues/325887
let(:query_count_multiplier) { 1 }
let(:query_count_multiplier) { 0 }
it_behaves_like 'an efficient database result'
end
......
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