diff options
| author | Stan Hu <stanhu@gmail.com> | 2019-08-13 19:04:30 +0300 |
|---|---|---|
| committer | Stan Hu <stanhu@gmail.com> | 2019-08-13 19:04:30 +0300 |
| commit | e658f9603c99ca6a8ef4c0292b2cdab2916fb09e (patch) | |
| tree | 57e0c03ea5dd676c7a53243bf3da20aefafc0ded /app/workers | |
| parent | 3702ab7317533896c7455357dd6643181666f22b (diff) | |
Only expire tag cache once per push
Previously each tag in a push would invoke the Gitaly `FindAllTags` RPC
since the tag cache would be invalidated with every tag.
We can eliminate those extraneous calls by expiring the tag cache once
in `PostReceive` and taking advantage of the cached tags.
Relates to https://gitlab.com/gitlab-org/gitlab-ce/issues/65795
Diffstat (limited to 'app/workers')
| -rw-r--r-- | app/workers/post_receive.rb | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/app/workers/post_receive.rb b/app/workers/post_receive.rb index d8dfbc0faf7..675fa1a73ca 100644 --- a/app/workers/post_receive.rb +++ b/app/workers/post_receive.rb @@ -44,6 +44,8 @@ class PostReceive # Expire the branches cache so we have updated data for this push post_received.project.repository.expire_branches_cache if post_received.includes_branches? + # We only need to expire tags once per push + post_received.project.repository.expire_caches_for_tags if post_received.includes_tags? post_received.enum_for(:changes_refs).with_index do |(oldrev, newrev, ref), index| service_klass = |