> -----Original Message----- > From: Simone Tripodi [mailto:simone.trip...@gmail.com] > Sent: Friday, October 29, 2010 20:34 > To: Commons Developers List > Subject: Re: [pool] Pool config vs. factory hierarchies. > > 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.
The current trunk code stores a Config reference, this patch does not change that. Gary > 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 > <simone.trip...@gmail.com> 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 > > <ggreg...@seagullsoftware.com> 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: ggreg...@seagullsoftware.com > >> Web: seagull.rocketsoftware.com > >> > >> > >> --------------------------------------------------------------------- > >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > >> For additional commands, e-mail: dev-h...@commons.apache.org > >> > >> > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org
