WIP: Put Rugged::FindCommit back into 11.5.7 EE (!9377) · Merge requests · GitLab.org / GitLab

Stan Hu requested to merge sh-graft-find-commit-11-5-7-ee into 11-5-stable-ee Jan 31, 2019

On one system, we saw Gitaly's FindCommit request launching many git cat-file calls per request, which caused unnecessary load when Gitaly is run atop of NFS.

By putting back Rugged, this eliminates many of the N+1 Gitaly queries called in FindCommit. The repository is loaded once, and all subsequent loads for commits are much cheaper.

To apply this patch:

curl -o /tmp/9377.patch https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/9377.diff
cd /opt/gitlab/embedded/service/gitlab-rails
patch -p1 < /tmp/9377.patch

Edited Jan 31, 2019 by Stan Hu

WIP: Put Rugged::FindCommit back into 11.5.7 EE

Merge request reports