On 10/4/07, William Tam <[EMAIL PROTECTED]> wrote:
> 1. In SedaEndpoint.process(Exchange exchange, AsyncCallback callback)
> method.   I wouldn't expect callback.done() to be invoked immediately
> when an exchange is enqueued.  (right?) But rather I would expect
> callbacks are enqueued with the exchanges so that when the
> SedaConsumer can invoke them after their corresponding exchanges are
> processed.

I don't agree.  I think seda should behave like a jms queue.. The
producer is done when the message gets enqueued.  If what you are
trying to do is to do async processing of a pipeline, you should use
the thread processor, like:
from(x).thread(5).to(y)


>
> 2. In the SedaConsumer.run() method.  I think the two checks for
> "!isStopping()" should be modified to: "!isStopping() &&
> !isStopped()".  Otherwise, the consumer's thread can run forever since
> it is possible for ServiceSupport.stop()  to change the stopping flag
> to true and back to false before the consumer's thread has a chance to
> check the stopping flag.

Good catch!  Will fix.

>
> Regards,
> William
>


-- 
Regards,
Hiram

Blog: http://hiramchirino.com

Reply via email to