Set size of connection pool greater than concurrency by default

[nsoufr]

Because of the nature of Upperkut, it needs a certain number of connections to work properly in terms of performance. And it needs even more connections when users increase concurrency option.

So by default, we need to implement a default pool size that is greater than the concurrency option and enable users to customize it when they want.

class MyWorker
  setup_upperkut do |config|
    config.strategy = Upperkut::Strategy.new(self, redis: { url: ENV['REDIS_URL'], size: 200 } )
  end
end

[lucaskds]

We need a better approach because we're passing self to the strategy but is possible to change the concurrency after the Strategy instantiation, like:

class MyWorker
  setup_upperkut do |config|
    config.strategy = Upperkut::Strategy.new(self, redis: { url: ENV['REDIS_URL'] } )
    config.concurrency = 30
  end
end

This will cause a config with size = 5 and concurrency = 30... 🚫

[nsoufr]

🤔

[nsoufr]

We need to benchmark other gems in order to make a decision here.