activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From OneWhoMikes <richmondm...@gmail.com>
Subject Re: MessageServlet reads too many messages - Fixed (patch included)
Date Fri, 26 Sep 2008 14:47:53 GMT

No Problem, glad I could help.

JIRA issue is here:
https://issues.apache.org/activemq/browse/AMQ-1955


--Mike



bsnyder wrote:
> 
> On Wed, Sep 24, 2008 at 9:27 PM, OneWhoMikes <richmondmike@gmail.com>
> wrote:
>>
>> I just noticed that the MessageServlet inside activemq-web will pull
>> maxMessages+1 messages off of the queue but only write a response
>> containing
>> maxMessages.
>>
>> After some debugging, I figured out that the problem is with the while
>> loop
>> that reads messages inside the doMessages method.  It always reads a
>> message
>> at the end of the loop, without first checking to see if maxMessages has
>> already been reached.
>>
>> I have attached a patchfile here that corrects the issue.
>> http://www.nabble.com/file/p19662481/patchfile.txt patchfile.txt
> 
> Thanks, Mike! Please create a JIRA issue and attach the patch to it
> for tracking purposes.
> 
> Bruce
> -- 
> perl -e 'print
> unpack("u30","D0G)U8V4\@4VYY9&5R\"F)R=6-E+G-N>61E<D\!G;6%I;\"YC;VT*"
> );'
> 
> Apache ActiveMQ - http://activemq.org/
> Apache Camel - http://activemq.org/camel/
> Apache ServiceMix - http://servicemix.org/
> 
> Blog: http://bruceblog.org/
> 
> 

-- 
View this message in context: http://www.nabble.com/MessageServlet-reads-too-many-messages---Fixed-%28patch-included%29-tp19662481p19689943.html
Sent from the ActiveMQ - Dev mailing list archive at Nabble.com.


Mime
View raw message