Re: [pool] Reusing Config

2010-10-28 Thread Simone Tripodi
Hi all guys, sorry to be late but I had to read all the resume before posting something useful :P So, briefly, just my 2 cents with the hope to contribute in a useful way: About the JMX support, that could involve also the whole design, I suggest on keeping out the Configuration but rather taking

Re: [pool] Reusing Config

2010-10-28 Thread James Carman
On Thu, Oct 28, 2010 at 6:50 AM, Simone Tripodi simone.trip...@gmail.com wrote: That's why I wouldn't follow the option #2; but please, explain me which are the side effects of that design, so I can avoid to repeat the same mistake in the future. If you make the config objects immutable, you

Re: [pool] Reusing Config

2010-10-28 Thread Simone Tripodi
Hi James, thanks, understood. I take in charge yet another cycle of refactoring, I'll ping you all when in trouble about something :) Have a nice day and thanks for the feedbacks! Simo http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Thu, Oct 28, 2010 at 2:12 PM, James Carman

Re: [pool] Reusing Config

2010-10-28 Thread Simone Tripodi
Hi again James, all, please review my last commit[1], I refactored the Config and related (Keyed)StackObjectPool(Factory), I tried to implements the concepts we've been discussing in this thread. If this fits to our vision, I can follow applying the refactor to Generic(Keyed)ObjectPool(Factory).

Re: [pool] Reusing Config

2010-10-28 Thread sebb
On 28 October 2010 15:16, Simone Tripodi simone.trip...@gmail.com wrote: Hi again James, all, please review my last commit[1], I refactored the Config and related (Keyed)StackObjectPool(Factory), I tried to implements the concepts we've been discussing in this thread. The public

Re: [pool] Reusing Config

2010-10-28 Thread Simone Tripodi
Hi Seb, thanks for your feedbacks. Please read my questions inline your comments The public StackObjectPoolConfig ctor repeats the settings available in the nested Builder. I'm not sure I see the point of having both. Or perhaps the ctor is supposed to be private? I'm sorry but I didn't

Re: [pool] Reusing Config

2010-10-28 Thread sebb
On 28 October 2010 18:50, Simone Tripodi simone.trip...@gmail.com wrote: Hi Seb, thanks for your feedbacks. Please read my questions inline your comments The public StackObjectPoolConfig ctor repeats the settings available in the nested Builder. I'm not sure I see the point of having both.

Re: [pool] Reusing Config

2010-10-28 Thread Simone Tripodi
Thanks a lot Seb, your suggestions are always appreciated :) Thanks for the sample, I see your troubles on ctor/builder, I think it makes much more sense when the number of arguments is very large, like the Generic(Keyed)ObjectPool(Factory). Do you have options to suggest? Thanks in advance.

Re: [pool] Reusing Config part 2

2010-10-26 Thread Simone Tripodi
...@seagullsoftware.com wrote: -Original Message- From: Phil Steitz [mailto:phil.ste...@gmail.com] Sent: Monday, October 25, 2010 08:50 To: Commons Developers List Subject: Re: [pool] Reusing Config part 2 On 10/25/10 11:26 AM, Gary Gregory wrote: Thank you for working through

Re: [pool] Reusing Config

2010-10-25 Thread Simone Tripodi
: [pool] Reusing Config Gary, Great work so far.  I'm checking out the diffs now, I'm gonna hack out some simple UML diffs, if only to wrap my head around it all. I'll upload the file to the issue once complete. BTW, I hope I didn't offend with the 'academic' comment, I most certainly did

Re: [pool] Reusing Config

2010-10-25 Thread Steven Siebert
] Sent: Thursday, October 21, 2010 18:08 To: Commons Developers List Subject: Re: [pool] Reusing Config Gary, Great work so far. I'm checking out the diffs now, I'm gonna hack out some simple UML diffs, if only to wrap my head around it all. I'll upload the file to the issue

Re: [pool] Reusing Config

2010-10-25 Thread Simone Tripodi
...@seagullsoftware.com wrote: -Original Message- From: Steven Siebert [mailto:smsi...@gmail.com] Sent: Thursday, October 21, 2010 18:08 To: Commons Developers List Subject: Re: [pool] Reusing Config Gary, Great work so far.  I'm checking out the diffs now, I'm gonna hack

Re: [pool] Reusing Config

2010-10-25 Thread Steven Siebert
...@gmail.com] Sent: Thursday, October 21, 2010 18:08 To: Commons Developers List Subject: Re: [pool] Reusing Config Gary, Great work so far. I'm checking out the diffs now, I'm gonna hack out some simple UML diffs, if only to wrap my head around it all. I'll upload

Re: [pool] Reusing Config

2010-10-25 Thread Simone Tripodi
.99soft.org/ On Fri, Oct 22, 2010 at 5:23 AM, Gary Gregory ggreg...@seagullsoftware.com wrote: -Original Message- From: Steven Siebert [mailto:smsi...@gmail.com] Sent: Thursday, October 21, 2010 18:08 To: Commons Developers List Subject: Re: [pool] Reusing Config Gary

Re: [pool] Reusing Config

2010-10-25 Thread Phil Steitz
/ On Fri, Oct 22, 2010 at 5:23 AM, Gary Gregory ggreg...@seagullsoftware.com wrote: -Original Message- From: Steven Siebert [mailto:smsi...@gmail.com] Sent: Thursday, October 21, 2010 18:08 To: Commons Developers List Subject: Re: [pool] Reusing Config Gary, Great work so far. I'm checking

Re: [pool] Reusing Config

2010-10-25 Thread Steven Siebert
:23 AM, Gary Gregory ggreg...@seagullsoftware.com wrote: -Original Message- From: Steven Siebert [mailto:smsi...@gmail.com] Sent: Thursday, October 21, 2010 18:08 To: Commons Developers List Subject: Re: [pool] Reusing Config Gary, Great work so far. I'm checking out the diffs

[pool] Reusing Config part 2

2010-10-25 Thread Gary Gregory
  -Original Message- From: Simone Tripodi [mailto:simone.trip...@gmail.com] Sent: Monday, October 25, 2010 05:36 To: Commons Developers List Subject: Re: [pool] Reusing Config Hi all mates, I updated the jira issue uploading my patch; it contains the configuration extraction and some code

Re: [pool] Reusing Config

2010-10-25 Thread Phil Steitz
: Steven Siebert [mailto:smsi...@gmail.com] Sent: Thursday, October 21, 2010 18:08 To: Commons Developers List Subject: Re: [pool] Reusing Config Gary, Great work so far. I'm checking out the diffs now, I'm gonna hack out some simple UML diffs, if only to wrap my head around it all. I'll upload

Re: [pool] Reusing Config part 2

2010-10-25 Thread Steven Siebert
-Original Message- From: Simone Tripodi [mailto:simone.trip...@gmail.com] Sent: Monday, October 25, 2010 05:36 To: Commons Developers List Subject: Re: [pool] Reusing Config Hi all mates, I updated the jira issue uploading my patch; it contains the configuration extraction

Re: [pool] Reusing Config part 2

2010-10-25 Thread Phil Steitz
Email: ggreg...@seagullsoftware.com Web: seagull.rocketsoftware.com -Original Message- From: Simone Tripodi [mailto:simone.trip...@gmail.com] Sent: Monday, October 25, 2010 05:36 To: Commons Developers List Subject: Re: [pool] Reusing Config Hi all mates, I updated the jira issue uploading

Re: [pool] Reusing Config

2010-10-25 Thread Phil Steitz
, October 21, 2010 18:08 To: Commons Developers List Subject: Re: [pool] Reusing Config Gary, Great work so far. I'm checking out the diffs now, I'm gonna hack out some simple UML diffs, if only to wrap my head around it all. I'll upload the file to the issue once complete. BTW, I hope I

Re: [pool] Reusing Config

2010-10-25 Thread James Carman
On Mon, Oct 25, 2010 at 12:25 PM, Phil Steitz phil.ste...@gmail.com wrote: I notice now what I missed on initial review of Simo's patch - the pool accessors now manage the config properties via persisted Config members.  I am OK with this, but it now means that the Config classes have to be

Re: [pool] Reusing Config

2010-10-25 Thread Phil Steitz
On 10/25/10 12:36 PM, James Carman wrote: On Mon, Oct 25, 2010 at 12:25 PM, Phil Steitzphil.ste...@gmail.com wrote: I notice now what I missed on initial review of Simo's patch - the pool accessors now manage the config properties via persisted Config members. I am OK with this, but it now

RE: [pool] Reusing Config part 2

2010-10-25 Thread Gary Gregory
-Original Message- From: Steven Siebert [mailto:smsi...@gmail.com] Sent: Monday, October 25, 2010 08:43 To: Commons Developers List Subject: Re: [pool] Reusing Config part 2 Gary, I tossed this around as well, and noted these fields as a possible promote to the Abstract

RE: [pool] Reusing Config part 2

2010-10-25 Thread Gary Gregory
-Original Message- From: Phil Steitz [mailto:phil.ste...@gmail.com] Sent: Monday, October 25, 2010 08:50 To: Commons Developers List Subject: Re: [pool] Reusing Config part 2 On 10/25/10 11:26 AM, Gary Gregory wrote: Thank you for working through this Simone. I would like

Re: [pool] Reusing Config

2010-10-21 Thread Phil Steitz
, Gary Gregory ggreg...@seagullsoftware.com wrote: -Original Message- From: Simone Tripodi [mailto:simone.trip...@gmail.com] Sent: Wednesday, October 20, 2010 22:41 To: Commons Developers List Subject: Re: [pool] Reusing Config Hi Gary! unfortunately the link replied with 404 code, can

RE: [pool] Reusing Config

2010-10-21 Thread Gary Gregory
-Original Message- From: Phil Steitz [mailto:phil.ste...@gmail.com] Sent: Thursday, October 21, 2010 06:29 To: Commons Developers List Subject: Re: [pool] Reusing Config On 10/21/10, Simone Tripodi simone.trip...@gmail.com wrote: it seems you've been doing a very good work

Re: [pool] Reusing Config

2010-10-21 Thread Steven Siebert
Steitz [mailto:phil.ste...@gmail.com] Sent: Thursday, October 21, 2010 06:29 To: Commons Developers List Subject: Re: [pool] Reusing Config On 10/21/10, Simone Tripodi simone.trip...@gmail.com wrote: it seems you've been doing a very good work, the only thing I *suggest

[pool] Reusing Config

2010-10-20 Thread Gary Gregory
Hi All: I think this came up recently. Any thoughts or plans on extracting the Config class out of GenericKeyedObjectPool and GenericObjectPool so it can be reused. The constants for default values could then also be moved to Config. Gary Gregory Senior Software Engineer Rocket Software 3340

RE: [pool] Reusing Config

2010-10-20 Thread Gary Gregory
: Wednesday, October 20, 2010 10:29 To: Commons Developers List Subject: [pool] Reusing Config Hi All: I think this came up recently. Any thoughts or plans on extracting the Config class out of GenericKeyedObjectPool and GenericObjectPool so it can be reused. The constants for default values

Re: [pool] Reusing Config

2010-10-20 Thread Simone Tripodi
: Wednesday, October 20, 2010 10:29 To: Commons Developers List Subject: [pool] Reusing Config Hi All: I think this came up recently. Any thoughts or plans on extracting the Config class out of GenericKeyedObjectPool and GenericObjectPool so it can be reused. The constants for default values

Re: [pool] Reusing Config

2010-10-20 Thread Gary Gregory
, October 20, 2010 10:29 To: Commons Developers List Subject: [pool] Reusing Config Hi All: I think this came up recently. Any thoughts or plans on extracting the Config class out of GenericKeyedObjectPool and GenericObjectPool so it can be reused. The constants for default values could

Re: [pool] Reusing Config

2010-10-20 Thread Simone Tripodi
, October 20, 2010 10:29 To: Commons Developers List Subject: [pool] Reusing Config Hi All: I think this came up recently. Any thoughts or plans on extracting the Config class out of GenericKeyedObjectPool and GenericObjectPool so it can be reused. The constants for default values could

Re: [pool] Reusing Config

2010-10-20 Thread Steven Siebert
...@seagullsoftware.com Web: seagull.rocketsoftware.com -Original Message- From: Gary Gregory [mailto:ggreg...@seagullsoftware.com] Sent: Wednesday, October 20, 2010 10:29 To: Commons Developers List Subject: [pool] Reusing Config Hi All: I think this came up recently

RE: [pool] Reusing Config

2010-10-20 Thread Gary Gregory
-Original Message- From: Steven Siebert [mailto:smsi...@gmail.com] Sent: Wednesday, October 20, 2010 12:51 To: Commons Developers List Subject: Re: [pool] Reusing Config I apologize for not getting the proposal for the MBean API out quite yet - needed some sleep last night

RE: [pool] Reusing Config

2010-10-20 Thread Gary Gregory
Email: ggreg...@seagullsoftware.com Web: seagull.rocketsoftware.com  -Original Message- From: Simone Tripodi [mailto:simone.trip...@gmail.com] Sent: Wednesday, October 20, 2010 12:22 To: Commons Developers List Subject: Re: [pool] Reusing Config sure, I always wait for feedbacks

Re: [pool] Reusing Config

2010-10-20 Thread Simone Tripodi
[mailto:simone.trip...@gmail.com] Sent: Wednesday, October 20, 2010 12:22 To: Commons Developers List Subject: Re: [pool] Reusing Config sure, I always wait for feedbacks before coding :P Cool expression Rambo through the code, that was the first time I read it and made me laugh :D All the best

RE: [pool] Reusing Config

2010-10-20 Thread Gary Gregory
Web: seagull.rocketsoftware.com  -Original Message- From: Simone Tripodi [mailto:simone.trip...@gmail.com] Sent: Wednesday, October 20, 2010 14:53 To: Commons Developers List Subject: Re: [pool] Reusing Config Hi, sorry for not having been clear, but in my previous email my

Re: [pool] Reusing Config

2010-10-20 Thread Simone Tripodi
Web: seagull.rocketsoftware.com -Original Message- From: Simone Tripodi [mailto:simone.trip...@gmail.com] Sent: Wednesday, October 20, 2010 14:53 To: Commons Developers List Subject: Re: [pool] Reusing Config Hi, sorry for not having been clear, but in my previous email my