Allow n+1s caused by avatar fetches on the project dashboard. See…

Allow n+1s caused by avatar fetches on the project dashboard. See https://gitlab.com/gitlab-org/gitlab-ce/issues/38261

Allow n+1s caused by avatar fetches on the project dashboard. See…
e4b69bc2 · Andrew Newdigate · 4d88f649 · e4b69bc2
Commit e4b69bc2 authored Sep 25, 2017 by Andrew Newdigate
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

repository.rb app/models/repository.rb +5 -2

No files found.
--- a/app/models/repository.rb
+++ b/app/models/repository.rb
@@ -534,8 +534,11 @@ class Repository
  cache_method :tag_count, fallback: 0
  def avatar
-    if tree = file_on_head(:avatar)
+    # n+1: https://gitlab.com/gitlab-org/gitlab-ce/issues/38327
-      tree.path
+    Gitlab::GitalyClient.allow_n_plus_1_calls do
+      if tree = file_on_head(:avatar)
+        tree.path
+      end
    end
  end
  cache_method :avatar