Skip to content

Fix DeleteUserWorker query limit

Andrew Evans requested to merge 464672-delete-user-worker-query-limit into master

What does this MR do and why?

Fix DeleteUserWorker query limit

Currently, query limiting is disabled for the DeleteUserWorker. This change turns query limiting back on.

If we ever reach the next 200 threshold in tests, then we would consider work to break up the DeleteUserWorker so that it does work in smaller chunks.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

How to set up and validate locally

  1. rspec spec/workers/delete_user_worker_spec.rb spec/controllers/admin/users_controller_spec.rb spec/services/groups/destroy_service_spec.rb spec/controllers/projects_controller_spec.rb

Related to #464672 (closed)

Edited by Thong Kuah

Merge request reports

Loading