[go: up one dir, main page]

Do not throttle enqueueing of BulkImports::EntityWorker

What does this MR do and why?

Mentions #357314 (closed)

This MR removes job enqueueing throttling of BulkImports::EntityWorker and gives back full control to Sidekiq to execute the jobs according to their priority and concurrency settings. The original reasoning behind this change was to make sure if multiple migrations happen at the same time we don't delay all other jobs in the shard, but I don't think this will happen. Let Sidekiq do its thing and remove a limit of 5 'started' entities at a time 😄

The end result is more entities will have state 'started' and the entity job will be enqueued, but it does not mean all of them wll be processed at the same time.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

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

Edited by George Koltsov

Merge request reports

Loading