diff options
author | Ahmad Sherif <me@ahmadsherif.com> | 2018-02-08 17:14:45 +0100 |
---|---|---|
committer | Ahmad Sherif <me@ahmadsherif.com> | 2018-02-08 19:20:46 +0100 |
commit | 8cdb3477b091e5c4ce08ef9f8caa65c7f5d925af (patch) | |
tree | e0d6332ddf4ce580b10433489924bac2a14e618c /lib | |
parent | 5edd94ae75c103d468b8b307e37321b92bde6080 (diff) | |
download | gitlab-ce-8cdb3477b091e5c4ce08ef9f8caa65c7f5d925af.tar.gz |
Remove allow_n_plus_1 from Git::Repository#branches_filter
Diffstat (limited to 'lib')
-rw-r--r-- | lib/gitlab/git/repository.rb | 19 |
1 files changed, 8 insertions, 11 deletions
diff --git a/lib/gitlab/git/repository.rb b/lib/gitlab/git/repository.rb index 6761fb0937a..5f014e43c6f 100644 --- a/lib/gitlab/git/repository.rb +++ b/lib/gitlab/git/repository.rb @@ -1614,17 +1614,14 @@ module Gitlab # Gitaly note: JV: Trying to get rid of the 'filter' option so we can implement this with 'git'. def branches_filter(filter: nil, sort_by: nil) - # n+1: https://gitlab.com/gitlab-org/gitlab-ce/issues/37464 - branches = Gitlab::GitalyClient.allow_n_plus_1_calls do - rugged.branches.each(filter).map do |rugged_ref| - begin - target_commit = Gitlab::Git::Commit.find(self, rugged_ref.target) - Gitlab::Git::Branch.new(self, rugged_ref.name, rugged_ref.target, target_commit) - rescue Rugged::ReferenceError - # Omit invalid branch - end - end.compact - end + branches = rugged.branches.each(filter).map do |rugged_ref| + begin + target_commit = Gitlab::Git::Commit.find(self, rugged_ref.target) + Gitlab::Git::Branch.new(self, rugged_ref.name, rugged_ref.target, target_commit) + rescue Rugged::ReferenceError + # Omit invalid branch + end + end.compact sort_branches(branches, sort_by) end |