On Mar 22, 2012, at 8:38 AM, AndyG wrote: > Ok, so I know this is me - but I think the test needs the fix? Opinions. > > Changed the timer to fire a sweep after defined period. I just don't see why > a sweep should occur immediately when the pool is created as it is unlikely > to need one?
Saw that change. The sweeper both adds and removes instances from the pool. If the pool is below the minimum, the sweeper will create the required instances via the Pool.Supplier. I had actually forgotten that detail myself. I think we have code in the stateless container to eagerly fill pools which we probably never needed. The only trick is we'd have to add code to wait for the pool to fill itself before returning from the Container.start() method. -David
