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 /doc/gitlab-basics/add-merge-request.md | |
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 'doc/gitlab-basics/add-merge-request.md')
0 files changed, 0 insertions, 0 deletions