Merge branch 'use-cached-project-count' into 'master'

Used cached value of project count to reduce DB load

Requires gitlab-org/gitlab-ce!5746

Possible fix for gitlab-com/infrastructure#303

See merge request !637

CHANGELOG-EE

@@ -6,6 +6,7 @@ v 8.11.0 (unreleased)
   - [Elastic][Fix] Commit search breaks for some URLs on gitlab-ce project
 
 v 8.10.5 (unreleased)
+  - Used cached value of project count to reduce DB load
 
 v 8.10.4
   - Fix available users in userselect dropdown when there is more than one userselect on the page. !604 (Rik de Groot)

app/models/concerns/elastic/repositories_search.rb

@@ -12,7 +12,7 @@ module Elastic
       end
 
       def self.repositories_count
-        Project.count
+        Project.cached_count
       end
 
       def client_for_indexing

app/models/project.rb

@@ -408,6 +408,12 @@ class Project < ActiveRecord::Base
 
       joins(join_body).reorder('join_note_counts.amount DESC')
     end
+
+    def cached_count
+      Rails.cache.fetch('total_project_count', expires_in: 5.minutes) do
+        Project.count
+      end
+    end
   end
 
   def repository_storage_path

app/views/admin/dashboard/index.html.haml

@@ -131,7 +131,7 @@
           %h4 Projects
           .data
             = link_to admin_namespaces_projects_path do
-              %h1= number_with_delimiter(Project.count)
+              %h1= number_with_delimiter(Project.cached_count)
             %hr
             = link_to('New Project', new_project_path, class: "btn btn-new")
       .col-sm-4