GitHub user franz1981 opened a pull request: https://github.com/apache/activemq-artemis/pull/1786
ARTEMIS-1616 OpenWire improvements It includes several improvements on OpenWire: - Avoided copy of CoreMessage when not needed and cached lambda on hot path - Refactored OpenWireMessageConverter::inbound in order to group help the JVM compile most used methods - Optimized SimpleString::split because heavily used into AddressImpl::new - Added existing queues cache to avoid multiple expensive AMQSession::checkAutoCreateQueue calls - used SimpleString on OpenWireMessageConverter to avoid translations on CoreMessage - cached Notification Destination check on AMQConsumer - Used SimpleString on AMQSession explicitly with HDR_DUPLICATE_DETECTION_ID and CONNECTION_ID_PROPERTY_NAME - Refactored toAMQMessage by grouping methods for a better readability You can merge this pull request into a Git repository by running: $ git pull https://github.com/franz1981/activemq-artemis open_wire_to_amqp Alternatively you can review and apply these changes as the patch at: https://github.com/apache/activemq-artemis/pull/1786.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1786 ---- commit 4d493c0eae35fb0be2e3e212eadcb657346ff8f9 Author: Francesco Nigro <nigro.fra@...> Date: 2018-01-16T13:24:32Z Avoided copy of CoreMessage when not needed and cached lambda on hot path commit e4f9a4f9a0bc3277e580a3c014ba7d422597b81e Author: Francesco Nigro <nigro.fra@...> Date: 2018-01-16T16:01:54Z Refactored inbound in order to group help the JVM compile most used methods commit 82d1cd8c5489c106b5ff9e22b264216988b7986e Author: Francesco Nigro <nigro.fra@...> Date: 2018-01-16T17:40:06Z Optimized SimpleString::split because heavily used into AddressImpl::new commit df18521298e8866d1e75e299336d25845b97f663 Author: Francesco Nigro <nigro.fra@...> Date: 2018-01-16T20:27:22Z Added existing queues cache to avoid multiple expensive checkCreateQueue calls commit 74eb72bffa7c723e714d4744fcc9e7207b5fe92a Author: Francesco Nigro <nigro.fra@...> Date: 2018-01-16T21:24:08Z used SimpleString to avoid translations on CoreMessage commit 5752e30a9363bc473bd52e3ae09c677687ad1ab2 Author: Francesco Nigro <nigro.fra@...> Date: 2018-01-16T23:37:24Z cached Notification Destination check on AMQConsumer commit 94c9de15fb729d9e80000294166cc9d48fdd0e09 Author: Francesco Nigro <nigro.fra@...> Date: 2018-01-17T07:29:28Z Used SimpleString explicitly with HDR_DUPLICATE_DETECTION_ID and CONNECTION_ID_PROPERTY_NAME commit 1f53625848ae10269c9748b4125848609fd3854b Author: Francesco Nigro <nigro.fra@...> Date: 2018-01-17T13:37:08Z Refactored toAMQMessage by grouping methods for a better readability ---- ---