> -----Original Message----- > From: [email protected] [mailto:[email protected]] On > Behalf Of James Carman > Sent: Friday, October 29, 2010 20:46 > To: Commons Developers List > Subject: Re: [pool] Pool config vs. factory hierarchies. > > If the config objects are immutable, then you can store the reference.
I thought we said that pools settings should be configurable. The current Config root class has setters. Are we saying that, yes, pools are configurable post-creation but not through config objects? Should config objects be cloned when passed in a constructor then? Gary > > On Fri, Oct 29, 2010 at 10:34 AM, Simone Tripodi > <[email protected]> wrote: > > Hi again Gary, > > the only thing I'm not sure about the patch - not blockin anyway, it > > can be modified later - is that one of the discussed requirement was > > not storing the config reference but rather copying the data. > > BTW I like the design!!! > > Simo > > > > http://people.apache.org/~simonetripodi/ > > http://www.99soft.org/ > > > > > > > > On Fri, Oct 29, 2010 at 4:28 PM, Simone Tripodi > > <[email protected]> wrote: > >> Hi Gary, > >> well done, it seems to me it is a very good work, +1 on applying this > patch!!! > >> Have a nice day, > >> Simo > >> > >> http://people.apache.org/~simonetripodi/ > >> http://www.99soft.org/ > >> > >> > >> > >> On Fri, Oct 29, 2010 at 3:55 PM, Gary Gregory > >> <[email protected]> wrote: > >>> Hi All, > >>> > >>> I see now in trunk that GenericKeyedObjectPoolConfig extends > GenericObjectPoolConfig, which I like. > >>> > >>> It seems that the next step would be for GenericKeyedObjectPoolFactory and > GenericObjectPoolFactory to share a common superclass. > >>> > >>> To see what I mean, look at the patch in > https://issues.apache.org/jira/browse/POOL-177. > >>> > >>> Beyond that, the idea would be to pull up the factory ivar. > >>> > >>> Gary Gregory > >>> Senior Software Engineer > >>> Rocket Software > >>> 3340 Peachtree Road, Suite 820 . Atlanta, GA 30326 . USA > >>> Tel: +1.404.760.1560 > >>> Email: [email protected] > >>> Web: seagull.rocketsoftware.com > >>> > >>> > >>> --------------------------------------------------------------------- > >>> To unsubscribe, e-mail: [email protected] > >>> For additional commands, e-mail: [email protected] > >>> > >>> > >> > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [email protected] > > For additional commands, e-mail: [email protected] > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
