[GitHub] [activemq-artemis] gaohoward opened a new pull request #2859: ARTEMIS-2513 Large message's copy may be interfered by other threads

[GitBox <gi...@apache.org>]

gaohoward opened a new pull request #2859: ARTEMIS-2513 Large message's copy may be interfered by other threads
URL: https://github.com/apache/activemq-artemis/pull/2859
 
 
   In LargeMessageImpl.copy(long) it need to open the underlying
   file in order to read and copy bytes into the new copied message.
   However there is a chance that another thread can come in and close
   the file in the middle, making the copy failed
   with "channel is null" error.
   
   This is happening in cases where a large message is sent to a jms
   topic (multicast address). During delivery it to multiple
   subscribers, some consumer is doing delivery and closed the
   underlying file after. Some other consumer is rolling back
   the messages and eventually move it to DLQ (which will call
   the above copy method). So there is a chance this bug being hit on.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services