Controller Projects::IssuesController#move executes more than 100 SQL queries (7 day: 2421 hits, average 210 queries)
This controller executes more than 100 SQL queries, the maximum number of
queries allowed as per MR
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/16466. The number of
queries should be reduced to less than 100 and the corresponding whitelisted
sections of code (Gitlab::QueryLimiting.whitelist('...')
) should be removed.
Implementation plan
- Refactor
Projects::IssuesController#move
so it doesn't call::Issues::UpdateService.new(project: project, current_user: current_user, params: { target_project: new_project }).execute(issue)
and callIssues::MoveService.new(project: project, current_user: current_user).execute(issue, target_project)
directly.✅ - Optimize updating notes and use bulk insert if possible
- Optimize award emoji clone process.
- Check what other steps are needed.
Edited by Michael Kozono