You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by cl...@apache.org on 2016/09/19 17:49:50 UTC

[1/2] activemq-artemis git commit: This closes #781

Repository: activemq-artemis
Updated Branches:
  refs/heads/master 1f5f45ca9 -> 501d127b3


This closes #781


Project: http://git-wip-us.apache.org/repos/asf/activemq-artemis/repo
Commit: http://git-wip-us.apache.org/repos/asf/activemq-artemis/commit/501d127b
Tree: http://git-wip-us.apache.org/repos/asf/activemq-artemis/tree/501d127b
Diff: http://git-wip-us.apache.org/repos/asf/activemq-artemis/diff/501d127b

Branch: refs/heads/master
Commit: 501d127b35c1674813947ff270ef17c350136814
Parents: 1f5f45c 6b8d522
Author: Clebert Suconic <cl...@apache.org>
Authored: Mon Sep 19 13:49:40 2016 -0400
Committer: Clebert Suconic <cl...@apache.org>
Committed: Mon Sep 19 13:49:40 2016 -0400

----------------------------------------------------------------------
 .../artemis/core/client/impl/ClientProducerImpl.java        | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------



[2/2] activemq-artemis git commit: ARTEMIS-715 messages could be sent to wrong queue

Posted by cl...@apache.org.
ARTEMIS-715 messages could be sent to wrong queue

In rare circumstances MessageProducer can send a message
to wrong queue


Project: http://git-wip-us.apache.org/repos/asf/activemq-artemis/repo
Commit: http://git-wip-us.apache.org/repos/asf/activemq-artemis/commit/6b8d5229
Tree: http://git-wip-us.apache.org/repos/asf/activemq-artemis/tree/6b8d5229
Diff: http://git-wip-us.apache.org/repos/asf/activemq-artemis/diff/6b8d5229

Branch: refs/heads/master
Commit: 6b8d52295ec078cd93c968915580614ec7caa1d8
Parents: 1f5f45c
Author: Dmitrii Tikhomirov <dt...@redhat.com>
Authored: Mon Sep 19 19:00:21 2016 +0200
Committer: Clebert Suconic <cl...@apache.org>
Committed: Mon Sep 19 13:49:40 2016 -0400

----------------------------------------------------------------------
 .../artemis/core/client/impl/ClientProducerImpl.java        | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/6b8d5229/artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientProducerImpl.java
----------------------------------------------------------------------
diff --git a/artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientProducerImpl.java b/artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientProducerImpl.java
index 107df64..baa59f5 100644
--- a/artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientProducerImpl.java
+++ b/artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientProducerImpl.java
@@ -266,7 +266,7 @@ public class ClientProducerImpl implements ClientProducerInternal {
             largeMessageSend(sendBlocking, msgI, theCredits, handler);
          }
          else {
-            sendRegularMessage(msgI, sendBlocking, theCredits, handler);
+            sendRegularMessage(sendingAddress, msgI, sendBlocking, theCredits, handler);
          }
       }
       finally {
@@ -274,7 +274,8 @@ public class ClientProducerImpl implements ClientProducerInternal {
       }
    }
 
-   private void sendRegularMessage(final MessageInternal msgI,
+   private void sendRegularMessage(final SimpleString sendingAddress,
+                                   final MessageInternal msgI,
                                    final boolean sendBlocking,
                                    final ClientProducerCredits theCredits,
                                    final SendAcknowledgementHandler handler) throws ActiveMQException {
@@ -290,7 +291,7 @@ public class ClientProducerImpl implements ClientProducerInternal {
 
       theCredits.acquireCredits(creditSize);
 
-      session.checkDefaultAddress(address);
+      session.checkDefaultAddress(sendingAddress);
 
       sessionContext.sendFullMessage(msgI, sendBlocking, handler, address);
    }
@@ -491,7 +492,7 @@ public class ClientProducerImpl implements ClientProducerInternal {
                msgI.putLongProperty(Message.HDR_LARGE_BODY_SIZE, deflaterReader.getTotalSize());
 
                msgI.getBodyBuffer().writeBytes(buff, 0, pos);
-               sendRegularMessage(msgI, sendBlocking, credits, handler);
+               sendRegularMessage(msgI.getAddress(), msgI, sendBlocking, credits, handler);
                return;
             }
             else {