arnoudja commented on PR #1851: URL: https://github.com/apache/activemq/pull/1851#issuecomment-4149839694
Would be good to have that AMQP fix in, let me know if I can help or you want me to create the pr. So far no luck reproducing the STOMP issue on my machine. Let's separate that from the AMQP issue. If it's a client side issue, we can solve it ourselves and if it turns out to be a bug in ActiveMQ I'll create a new pr. The more substantial solution sounds like a good idea to me. There might be some first steps without much impact. Making getText() in the ActiveMQTextMessage class just return the decoded content without storing it looks like an option. It would make subsequent calls to getText slower compared to the current situation but that can be solved client side by just reusing the result. Taking it a step further could remove the text field completely, with setText using the storeContent code directly, but the impact of that is a bit harder to see for me. -- 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
