Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-19 Thread bvahdat
Like the one here: http://svn.apache.org/viewvc?view=revision&revision=1159323 @Christian please don't blame me posting here as I simply couldn't stop myself commenting on this. -- View this message in context: http://camel.465427.n5.nabble.com/HEADS-UP-Adjustments-to-ExecutorSer

Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-19 Thread Claus Ibsen
On Fri, Aug 19, 2011 at 2:22 PM, Christian Schneider wrote: > Committed the changes. > > I even built before I committed to show extra professionalism :-) > Ah that is maybe too professional. Sometimes you have to leave easy mistakes in for others to fix :) > Christian > > Am 19.08.2011 10:00,

Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-19 Thread Christian Schneider
Committed the changes. I even built before I committed to show extra professionalism :-) Christian Am 19.08.2011 10:00, schrieb Claus Ibsen: The latest commits look good. Only a few cosmetic changes that would be nice to do - Add missing/fix javadoc to ExecutorServiceManager (its a SPI interf

Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-19 Thread Claus Ibsen
The latest commits look good. Only a few cosmetic changes that would be nice to do - Add missing/fix javadoc to ExecutorServiceManager (its a SPI interface so full javadoc shows professionalism) - Add missing/fix javadoc to ThreadPoolFactory (its a SPI interface so full javadoc shows professionali

Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-18 Thread Claus Ibsen
On Thu, Aug 18, 2011 at 2:07 PM, Christian Schneider wrote: > Am 18.08.2011 13:50, schrieb Claus Ibsen: >> >> On Thu, Aug 18, 2011 at 1:25 PM, Christian Schneider >>  wrote: >>> >>> Hi Claus, >>> >>> I just saw that you also changed the ThreadPoolFactory interface to not >>> use >>> the profiles

Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-18 Thread Claus Ibsen
On Thu, Aug 18, 2011 at 2:07 PM, Christian Schneider wrote: > Am 18.08.2011 13:50, schrieb Claus Ibsen: >> >> On Thu, Aug 18, 2011 at 1:25 PM, Christian Schneider >>  wrote: >>> >>> Hi Claus, >>> >>> I just saw that you also changed the ThreadPoolFactory interface to not >>> use >>> the profiles

Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-18 Thread Christian Schneider
Am 18.08.2011 13:50, schrieb Claus Ibsen: On Thu, Aug 18, 2011 at 1:25 PM, Christian Schneider wrote: Hi Claus, I just saw that you also changed the ThreadPoolFactory interface to not use the profiles and instead have several methods again. This completely defeats the purpose of having a smal

Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-18 Thread Claus Ibsen
On Thu, Aug 18, 2011 at 1:50 PM, Claus Ibsen wrote: > On Thu, Aug 18, 2011 at 1:25 PM, Christian Schneider > wrote: >> Hi Claus, >> >> I just saw that you also changed the ThreadPoolFactory interface to not use >> the profiles and instead have several methods again. This completely defeats >> the

Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-18 Thread Claus Ibsen
On Thu, Aug 18, 2011 at 1:25 PM, Christian Schneider wrote: > Hi Claus, > > I just saw that you also changed the ThreadPoolFactory interface to not use > the profiles and instead have several methods again. This completely defeats > the purpose of having a small factory interface. > I will revert

Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-18 Thread Christian Schneider
Hi Claus, I just saw that you also changed the ThreadPoolFactory interface to not use the profiles and instead have several methods again. This completely defeats the purpose of having a small factory interface. I will revert that back. I worked on these changes quite a long time so the fact

Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-18 Thread Christian Schneider
Hi Claus, sorry for getting back this is so late but I had to work on other projects in the mean time. 2.9 Your ideas for camel 3.0 are rather small changes that can mostly be done in 2.9. > We could move ThreadPoolProfile from org.apache.camel.spi to org.apache.camel and have it in the ro

Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-13 Thread Claus Ibsen
I took the liberty of adding some notes to the Camel 3.0 roadmap https://cwiki.apache.org/confluence/display/CAMEL/Camel+3.0+-+Roadmap See the section - Improvements to ThreadPoolProfile for thread management In fact I think we could possible make a compromise with the changes from Christian and

Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-13 Thread Claus Ibsen
On Sat, Aug 13, 2011 at 11:20 AM, Christian Schneider wrote: > I am -1 on your last changes (CAMEL-4298). You effectively rolled back most > of my changes from CAMEL-4244. > > I intend to make the ThreadPoolProfile the main configuration method for > thread pools. This makes working with defaults

Re: [HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-13 Thread Christian Schneider
I am -1 on your last changes (CAMEL-4298). You effectively rolled back most of my changes from CAMEL-4244. I intend to make the ThreadPoolProfile the main configuration method for thread pools. This makes working with defaults much more natural than with the more JDK style API. Your change pul

[HEADS UP] - Adjustments to ExecutorServiceManager on trunk

2011-08-12 Thread Claus Ibsen
Hi Recently the ExecutorServiceStrategy had a bigger refactor (https://issues.apache.org/jira/browse/CAMEL-4244) Such as renaming the class to a better name ExecutorServiceManager. Also introducing a ThreadPoolFactory, ThreadFactory among others. The ThreadPoolFactory is a great addition, which ma