[prev in list] [next in list] [prev in thread] [next in thread] 

List:       activemq-dev
Subject:    [GitHub] CNNJYB opened a new pull request #2533: ARTEMIS-2244 checkDepage method placed outside CRIT
From:       GitBox <git () apache ! org>
Date:       2019-01-31 9:22:54
Message-ID: 154892657484.20501.15112715605180919011.gitbox () gitbox ! apache ! org
[Download RAW message or body]

CNNJYB opened a new pull request #2533: ARTEMIS-2244 checkDepage method placed \
                outside CRITICAL_DELIVER avoid critical analyzer timeout
URL: https://github.com/apache/activemq-artemis/pull/2533
 
 
   We found server crash becauseof critical analyzer timeout, thread dump can be seen \
in attachment.	  Suppose that there is a topic t with two subscriber ta and tb. Some \
messages were routed to subscriber ta, not to tb. When a consumer that subscribe tb \
is created and send ConsumerCredits to server, ServerConsumerImpl will call \
promptDelivery method, then forceDelivery() -> messageQueue.deliverAsync() -> \
deliverRunner will executed in the pageSubscription's executor(step1), then \
checkDepage(step2) -> pageIterator.hasNext(synchronized method) -> next -> moveNext, \
moveNext method will iterate through queue until find a matching message. At this \
time(step1), DeliverRunner call deliver method -> checkDepage -> \
pageIterator.hasNext, this step blocked on CursorIterator which was locked by step2, \
then critical analyzer timeout.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services


[prev in list] [next in list] [prev in thread] [next in thread]