diff options
| author | Robert Gemmell <robbie@apache.org> | 2011-04-04 14:35:59 +0000 |
|---|---|---|
| committer | Robert Gemmell <robbie@apache.org> | 2011-04-04 14:35:59 +0000 |
| commit | da5b1ae89dd0fd095c3973d07ac84dba98920b85 (patch) | |
| tree | 742ea747fcd7e3499edcc9b31324261f435d5fe0 /java | |
| parent | ac1fd87b2a48c277117c5c7a4a70ccef35f99acc (diff) | |
| download | qpid-python-da5b1ae89dd0fd095c3973d07ac84dba98920b85.tar.gz | |
QPID-3183: some minor improvements to the recently seperated QueueRunner class, based on review feedback from Keith Wall for QPID-3167
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@1088639 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'java')
| -rw-r--r-- | java/broker/src/main/java/org/apache/qpid/server/queue/QueueRunner.java | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/java/broker/src/main/java/org/apache/qpid/server/queue/QueueRunner.java b/java/broker/src/main/java/org/apache/qpid/server/queue/QueueRunner.java index 44b7c95535..7e1d57e205 100644 --- a/java/broker/src/main/java/org/apache/qpid/server/queue/QueueRunner.java +++ b/java/broker/src/main/java/org/apache/qpid/server/queue/QueueRunner.java @@ -27,13 +27,18 @@ import org.apache.qpid.server.logging.actors.CurrentActor; import org.apache.qpid.server.queue.QueueRunner; import org.apache.qpid.server.queue.SimpleAMQQueue; - +/** + * QueueRunners are Runnables used to process a queue when requiring + * asynchronous message delivery to subscriptions, which is necessary + * when straight-through delivery of a message to a subscription isn't + * possible during the enqueue operation. + */ public class QueueRunner implements ReadWriteRunnable { private static final Logger _logger = Logger.getLogger(QueueRunner.class); - private String _name; - private SimpleAMQQueue _queue; + private final String _name; + private final SimpleAMQQueue _queue; public QueueRunner(SimpleAMQQueue queue, long count) { @@ -53,7 +58,7 @@ public class QueueRunner implements ReadWriteRunnable } catch (AMQException e) { - _logger.error(e); + _logger.error("Exception during asynchronous delivery by " + _name, e); } finally { |