diff options
| author | Yorick Peterse <yorickpeterse@gmail.com> | 2016-10-04 17:22:58 +0200 |
|---|---|---|
| committer | Yorick Peterse <yorickpeterse@gmail.com> | 2016-10-04 19:41:37 +0200 |
| commit | c9bcfc631a79db97203d51d7c221a7549ba65adf (patch) | |
| tree | 01a8be8de90c498e916bc9f6d8c2bc7478e89226 /generator_templates | |
| parent | 7887a3dafb6902cc39c831e307c48faf49f80b51 (diff) | |
| download | gitlab-ce-c9bcfc631a79db97203d51d7c221a7549ba65adf.tar.gz | |
Remove lease from Event#reset_project_activityremove-reset-project-activity-lease
Per GitLab.com's performance metrics this method could take up to 5
seconds of wall time to complete, while only taking 1-2 milliseconds of
CPU time. Removing the Redis lease in favour of conditional updates
allows us to work around this.
A slight drawback is that this allows for multiple threads/processes to
try and update the same row. However, only a single thread/process will
ever win since the UPDATE query uses a WHERE condition to only update
rows that were not updated in the last hour.
Fixes gitlab-org/gitlab-ce#22473
Diffstat (limited to 'generator_templates')
0 files changed, 0 insertions, 0 deletions
