Remove unnecessary ignore for Gitaly N+1 (!20393) · Merge requests · GitLab.org / GitLab FOSS

Sean McGivern requested to merge remove-unnecessary-ignore-for-gitaly-n-plus-one into master Jul 05, 2018

First commit: make Gitaly N+1 backtraces better by reusing https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/19894.

Second commit: the problem here was that project isn't actually an attribute of NotificationSetting! It has a polymorphic source instead. So this was creating 10 projects instead of 5.

This is not urgent to review at all 🙂

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/47594.

Remove unnecessary ignore for Gitaly N+1

Merge request reports