pradeep85841 commented on PR #1659: URL: https://github.com/apache/activemq/pull/1659#issuecomment-3910178854
Removed synchronized keywords: Unit testing confirmed that the issue was not concurrent access to the same object reference, but rather a state-transition gap during memory reduction. State Safeguard in clearUnMarshalledState(): Added a check to ensure storeContent() is called if the message only exists in text form before the unmarshaled state is cleared. This prevents a race condition where both text and content could briefly be null. Idempotency: Since storeContent() internally checks for content == null, this approach is safe even if triggered multiple times, avoiding redundant marshaling overhead. -- 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
