NP Martin.  Sorry someone could not get back to you quicker.

Alex

On Jan 29, 2008 7:18 AM, Martin Alderson <
[EMAIL PROTECTED]> wrote:

> Hi all,
>
> I've just noticed that the latest trunk has already combined
> UnorderedExecutorFilter and ExecutorFilter.
>
> Please ignore my original message - I hope I didn't waste anyones time!
>
> Thanks,
>
> Martin
>
>
>
> >>> "Martin Alderson" <[EMAIL PROTECTED]> 28/01/2008
> 17:35 >>>
> Hi all,
>
> I've just spent a while trying to figure out the threading behaviour of
> MINA in the ApacheDS replication service.
>
> Eventually I found that it was using ThreadModel and that this is now not
> recommended.  After investigation into adding an ExecutorFilter to the end
> of the IoFilter chain I found that it was only using one thread even though
> I had specified a ThreadPoolExecutor.
>
> I see that MINA 2 has an UnorderedExecutorFilter which allows multiple
> threads to process the incoming messages simultaneously, but do we need to
> have an ExecutorFilter at all?  Couldn't we just use a single threaded
> executor with the UnorderedExecutorFilter like
> Executors.newSingleThreadExecutor()?
>
> Personally I think we should just replace ExecutorFilter with
> UnorderedExecutorFilter.  The current behaviour seems very misleading.  If
> this can't be done then perhaps we could update the documentation
> (specifically http://mina.apache.org/configuring-thread-model.html) to
> make it clear that ExecutorFilter will not make use of a ThreadPoolExecutor?
>
> Please correct me if I'm wrong with any of this - this is the first time
> I've even seen the Executor class :)
>
> Thanks,
>
> Martin
>
>
>
>

Reply via email to