Commit 75c3dc9e authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Merge branch 'sidekiq_gc_start' into 'master'

Run 'GC.start' after every EmailsOnPushWorker job

This change removed a lot of large peaks of the Sidekiq memory growth graph on gitlab.com.

See merge request !1257
parents 66f0c279 c10f6180
...@@ -2,6 +2,7 @@ v 7.5.0 ...@@ -2,6 +2,7 @@ v 7.5.0
- API: Add support for Hipchat (Kevin Houdebert) - API: Add support for Hipchat (Kevin Houdebert)
- Add time zone configuration on gitlab.yml (Sullivan Senechal) - Add time zone configuration on gitlab.yml (Sullivan Senechal)
- Fix LDAP authentication for Git HTTP access - Fix LDAP authentication for Git HTTP access
- Run 'GC.start' after every EmailsOnPushWorker job
- Fix LDAP config lookup for provider 'ldap' - Fix LDAP config lookup for provider 'ldap'
- Drop all sequences during Postgres database restore - Drop all sequences during Postgres database restore
- Project title links to project homepage (Ben Bodenmiller) - Project title links to project homepage (Ben Bodenmiller)
......
...@@ -21,5 +21,8 @@ class EmailsOnPushWorker ...@@ -21,5 +21,8 @@ class EmailsOnPushWorker
recipients.split(" ").each do |recipient| recipients.split(" ").each do |recipient|
Notify.repository_push_email(project_id, recipient, author_id, branch, compare).deliver Notify.repository_push_email(project_id, recipient, author_id, branch, compare).deliver
end end
ensure
compare = nil
GC.start
end end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment