BigW Consortium Gitlab

Skip to content

G
gitlab-ce
Commit d55d2256 by Douwe Maan Committed by Rémy Coutable
Options

Merge branch 'remove-sidekiq-backup-ar-threads' into 'master'

Don't use backup AR connections for Sidekiq Closes #27713 See merge request !9001
parent feab7e35
Showing with 5 additions and 3 deletions
changelogs/unreleased/remove-sidekiq-backup-ar-threads.yml 0 → 100644
---
title: Don't use backup Active Record connections for Sidekiq
merge_request:
author:
config/initializers/sidekiq.rb
......@@ -36,11 +36,9 @@ Sidekiq.configure_server do |config|
Gitlab::SidekiqThrottler.execute!
# Database pool should be at least `sidekiq_concurrency` + 2
# For more info, see: https://github.com/mperham/sidekiq/blob/master/4.0-Upgrade.md
config = ActiveRecord::Base.configurations[Rails.env] ||
Rails.application.config.database_configuration[Rails.env]
config['pool'] = Sidekiq.options[:concurrency] + 2
config['pool'] = Sidekiq.options[:concurrency]
ActiveRecord::Base.establish_connection(config)
Rails.logger.debug("Connection Pool size for Sidekiq Server is now: #{ActiveRecord::Base.connection.pool.instance_variable_get('@size')}")
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment