Skip to content

Re-enable RequestStore by default in Sidekiq

Stan Hu requested to merge sh-enable-request-store-sidekiq into master

!20291 (merged) accidentally made the RequestStore Sidekiq middleware disabled by default instead of enabled by default. This will cause a performance regression in Sidekiq jobs. For example, the Markdown cache will not be used.

For now, we default the REQUEST_STORE environment variable to 1. In the future, we should consider just dropping this environment variable.

Discovered in !19759 (comment 250602320)

Edited by Stan Hu

Merge request reports