Clear BatchLoader context between Sidekiq jobs

This commit is contained in:
Douwe Maan 2018-11-22 17:28:09 +01:00 committed by Stan Hu
parent ba9eeea4ee
commit 1b8a791829
3 changed files with 19 additions and 0 deletions

View File

@ -0,0 +1,5 @@
---
title: Clear BatchLoader context between Sidekiq jobs
merge_request: 23308
author:
type: fixed

View File

@ -20,6 +20,7 @@ Sidekiq.configure_server do |config|
chain.add Gitlab::SidekiqMiddleware::ArgumentsLogger if ENV['SIDEKIQ_LOG_ARGUMENTS'] && !enable_json_logs
chain.add Gitlab::SidekiqMiddleware::Shutdown
chain.add Gitlab::SidekiqMiddleware::RequestStoreMiddleware unless ENV['SIDEKIQ_REQUEST_STORE'] == '0'
chain.add Gitlab::SidekiqMiddleware::BatchLoader
chain.add Gitlab::SidekiqStatus::ServerMiddleware
end

View File

@ -0,0 +1,13 @@
# frozen_string_literal: true
module Gitlab
module SidekiqMiddleware
class BatchLoader
def call(worker, job, queue)
yield
ensure
::BatchLoader::Executor.clear_current
end
end
end
end