diff options
| author | Robert Gemmell <robbie@apache.org> | 2012-04-06 10:51:21 +0000 |
|---|---|---|
| committer | Robert Gemmell <robbie@apache.org> | 2012-04-06 10:51:21 +0000 |
| commit | 32e3df31c2dacb8af8af198c82881f42fee285dc (patch) | |
| tree | fdff5bc3322c97cb1b41df9951e7a74c0d775105 /qpid/java/test-profiles/Excludes | |
| parent | f41acf6529950afc916382754901c65cb0aca23c (diff) | |
| download | qpid-python-32e3df31c2dacb8af8af198c82881f42fee285dc.tar.gz | |
QPID-3911: Fix deadlock on concurrent invocation of MessageConsumer#close() and Session#rollback() from consumer MessageListener
This patch contains the following changes:
- Add synchronization on AMSession#_messageDeliveryLock into MessageConsumer#close() in order to block until message listener in progress has completed(as required in JMS javadoc for MessageConsumer#close()).
- Change the session dispatcher to stop messages delivery into consumer local message queue if the consumer in the process of closing. This eliminates the need to stop the dispatcher on rejecting pending messages for closing consumer.
- Remove the synchronization on the dispatcher lock from AMQSession.Dispatcher#rejectPending and code to stop the dispatcher, as we are synchronizing on the deliveryLock now and incoming messages are not dispatched into closing consumers anymore.
- Add a system test to reproduce the deadlock and verify its resolution.
Applied patch from Oleksandr Rudyy <orudyy@gmail.com>
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1310275 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'qpid/java/test-profiles/Excludes')
0 files changed, 0 insertions, 0 deletions