Skip to content
This repository has been archived by the owner on Nov 8, 2022. It is now read-only.

Commit

Permalink
Fixes #910
Browse files Browse the repository at this point in the history
Pool eligibility was not calculated properly for concurrency count > 1
  • Loading branch information
marcin-krolik committed May 12, 2016
1 parent 1d87e72 commit 079ba51
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions control/strategy/pool.go
Original file line number Diff line number Diff line change
Expand Up @@ -270,8 +270,9 @@ func (p *pool) Eligible() bool {
return false
}

should := p.SubscriptionCount() / p.concurrencyCount
if should > p.Count() && should <= p.max {
should := p.SubscriptionCount() > p.concurrencyCount*p.Count()
// Check if pool is eligible and number of plugins is less than maximum allowed
if should && p.Count() < p.max {
return true
}

Expand Down

0 comments on commit 079ba51

Please sign in to comment.