Skip to content

Use ETag to improve performance of issue notes polling

username-removed-378947 requested to merge etag-notes-polling into master

What does this MR do?

It avoids calls to the database when we poll for issue notes by implementing the proposal in https://gitlab.com/gitlab-org/gitlab-ce/issues/26926.

Are there points in the code the reviewer needs to double check?

  • regular expression in the middleware
  • cache expiry time

Why was this MR needed?

Notes polling generates high load on GitLab.com

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/27582

Merge request reports