Merge branch 'faster-rev-list' into 'master'

Faster rev list Since rugged does not support path filtering yet it affects GitLab performance. So we use `git rev-list` as temporary but fast and reliable solution for looking last commit of tree Fixes #1622 See merge request !1127

Merge branch 'faster-rev-list' into 'master'
2ebd2d41 · Dmitriy Zaporozhets · 0e8e9e5d · 44bc5fe5 · 2ebd2d41
Commit 2ebd2d41 authored Sep 29, 2014 by Dmitriy Zaporozhets
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

repository.rb app/models/repository.rb +3 -1

No files found.
--- a/app/models/repository.rb
+++ b/app/models/repository.rb
@@ -223,7 +223,9 @@ class Repository
  end

  def last_commit_for_path(sha, path)
-    commits(sha, path, 1).last
+    args = %W(git rev-list --max-count 1 #{sha} -- #{path})
+    sha = Gitlab::Popen.popen(args, path_to_repo).first.strip
+    commit(sha)
  end

  # Remove archives older than 2 hours