Skip to content

Fine tune over limit email notification worker

Doug Stull requested to merge 413547-update-notification-query into master

What does this MR do and why?

Fine tune over limit email notification worker as current method times out for sql statements - see https://gitlab.com/gitlab-org/gitlab/-/issues/413547#note_1422454511

As per !118906 (comment 1420459287)

  • Remove skip locked since we only use one worker with a 24hr buffer
  • Use read replica for reads
  • load the namespace_details in a separate query

Database

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #413547

Edited by Doug Stull

Merge request reports