Update documentation for worker removal
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.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by Kasia Misirli