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

----


---

Reply via email to