activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alex Dean <a...@crackpot.org>
Subject Re: need review on ajax change
Date Mon, 14 Feb 2011 14:24:21 GMT
Thanks for taking a look!

On Feb 14, 2011, at 4:32 AM, Gary Tully wrote:

> patch looks good.
> 
> On 13 February 2011 03:17, Alex Dean <alex@crackpot.org> wrote:
> 
>> I attached a patch to fix https://issues.apache.org/jira/browse/AMQ-3123.
>> 
>> If a continuation is not available when AjaxListener receives a message, it
>> buffers the message in a linked list for later delivery.
>> (MessageListenerServlet checks for any messages buffered in this way.)  The
>> problem is that AjaxListener isn't currently maintaining the
>> message<->consumer association, so some messages get associated with the
>> wrong callback function when they're sent back to the web browser.
>> 
>> The patch I attached fixes this problem, and I'd appreciate a review before
>> checking it in.
>> 
>> thanks,
>> alex
> 
> 
> 
> 
> -- 
> http://blog.garytully.com
> http://fusesource.com


Mime
View raw message