Skip to content

Implement more subscribers for MR approved event

Patrick Bajao requested to merge 10869-async-notes-todos-approval into master

What does this MR do and why?

This is part of the effort to reduce the SQL queries executed by the following subscribers to MergeRequests::ApprovedEvent:

  • CreateApprovalNoteWorker
  • ResolveTodosAfterApprovalWorker
  • ExecuteApprovalHooksWorker

This is behind async_after_approval feature flag.

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 #10869 (closed)

Edited by Patrick Bajao

Merge request reports