clebertsuconic commented on code in PR #5867:
URL: https://github.com/apache/activemq-artemis/pull/5867#discussion_r2269924061


##########
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientProducerImpl.java:
##########
@@ -411,6 +411,8 @@ private void largeMessageSendStreamed(final boolean 
sendBlocking,
       DeflaterReader deflaterReader = null;
 
       if (session.isCompressLargeMessages()) {
+         // We need to change properties the compressed message as we send it
+         msgI = msgI.copy();

Review Comment:
   @AntonRoskvist this is basically the fix for your negative positioning on 
the buffer. The body of the message is changed during send... and subsequent 
sends are getting lost.
   
   With this property the changes are now made on a copy.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org
For additional commands, e-mail: gitbox-h...@activemq.apache.org
For further information, visit: https://activemq.apache.org/contact


Reply via email to