Skip to content

Update documentation for worker removal

Kasia Misirli requested to merge km/Update_docs_worker_removal into master

What does this MR do and why?

Recently merged worker removal MR has caused an incident and it was reverted. In discover, two more cases of such incident were flushed out. This MR is updating the docs to hopefully prevent another incident from happening, until another solution will be found.

We want to advise users to always use disable_ddl_transaction! while using sidekiq_remove_jobs method, as we had multiple production incidents due to idle in transaction timeout.

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 Kasia Misirli

Merge request reports