summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHannes Rosenögger <123haynes@gmail.com>2015-03-13 19:16:42 +0000
committerHannes Rosenögger <123haynes@gmail.com>2015-03-13 19:16:42 +0000
commit07f9a3f928d39accf876d052b265844f74130099 (patch)
tree305322e406e3dda96e81a918f71f53a599d9fb6f
parent4803675190833cdf7e83558a32b2f2ea3283dce0 (diff)
parent9bb5986d4afff761ddc344413fdb34babb4c9d70 (diff)
downloadgitlab-ce-07f9a3f928d39accf876d052b265844f74130099.tar.gz
Merge branch 'follow-on-mr376' into 'master'
Follow on to MR 376 Follow-on to MR 376 from @haynes . Remove unnecessary fetch of commit messages for initial push. This will reduce the memory usage significantly. See merge request !387
-rw-r--r--CHANGELOG1
-rw-r--r--app/services/git_push_service.rb3
2 files changed, 1 insertions, 3 deletions
diff --git a/CHANGELOG b/CHANGELOG
index 5ccedcbc8c5..faf74ff2555 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -2,6 +2,7 @@ Please view this file on the master branch, on stable branches it's out of date.
v 7.9.0 (unreleased)
- Update documentation for object_kind field in Webhook push and tag push Webhooks (Stan Hu)
+ - Remove unnecessary fetch of commit messages on initial push (Perforce Software)
- Fix broken email images (Hannes Rosenögger)
- Fix mass SQL statements on initial push (Hannes Rosenögger)
- Add tag push notifications and normalize HipChat and Slack messages to be consistent (Stan Hu)
diff --git a/app/services/git_push_service.rb b/app/services/git_push_service.rb
index 4885e1b2fc5..e4c30dd6dfc 100644
--- a/app/services/git_push_service.rb
+++ b/app/services/git_push_service.rb
@@ -29,9 +29,6 @@ class GitPushService
elsif push_to_new_branch?(ref, oldrev)
# Re-find the pushed commits.
if is_default_branch?(ref)
- # Initial push to the default branch. Take the full history of that branch as "newly pushed".
- @push_commits = project.repository.commits(newrev)
-
# Set protection on the default branch if configured
if (current_application_settings.default_branch_protection != PROTECTION_NONE)
developers_can_push = current_application_settings.default_branch_protection == PROTECTION_DEV_CAN_PUSH ? true : false