Remove unnecessary fetch of commit messages for initial push. This will reduce...

Remove unnecessary fetch of commit messages for initial push. This will reduce the memory usage significantly.

CHANGELOG

 Please view this file on the master branch, on stable branches it's out of date.
 
 v 7.9.0 (unreleased)
+  - 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)

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