cshannon opened a new pull request, #2209: URL: https://github.com/apache/activemq/pull/2209
This is a small follow on to #2188 to simply some of the sync by moving to the method level and also adding sync in a spot or 2 that got missed. This commit also updates the race condition test to add all the other message types to show that they do not suffer from the same race condition with null bodies either. As noted in #2188, only text message needs sync because the other message types do not null out the marshalled content when unmarshalling. Properties are also ok because unmarshalling properties do not clear the marshalled properties either (unless told to at specific spots in the broker when reduceMemoryFootPrint is true) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information, visit: https://activemq.apache.org/contact
