activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From michaelandrepearce <>
Subject [GitHub] activemq-artemis pull request #1757: ARTEMIS-1586 Reduce GC pressure due to ...
Date Mon, 08 Jan 2018 19:19:09 GMT
GitHub user michaelandrepearce opened a pull request:

    ARTEMIS-1586 Reduce GC pressure due to String allocations on Core protocol

    @franz1981 i think this is cleaner merge of the two.

You can merge this pull request into a Git repository by running:

    $ git pull ARTEMIS-1586-FRANZ

Alternatively you can review and apply these changes as the patch at:

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1757
commit 4ab1fea88610e46381141223479aec97e7fa04ac
Author: Francesco Nigro <nigro.fra@...>
Date:   2018-01-04T14:22:05Z

    ARTEMIS-1586 Reduce GC pressure due to String allocations on Core protocol
    The commit contains:
    - a general purpose interner implementation
    - StringValue/SimpleString internrs specializations
    - TypedProperties keys/values string interning for SessionSendMessage decoding

commit 794e56da99456e891284709b790bf8144d65b0f9
Author: Michael André Pearce <michael.andre.pearce@...>
Date:   2018-01-08T18:45:18Z

    ARTEMIS-1586 Refactor to make more generic
    * Move byte util code into ByteUtil
    * Re-use the new equals method in SimpleString
    * Apply same pools/interners to client decode
    * Create String to SimpleString pools/interners for property access via String keys (producer
and consumer benefits)
    * Lazy init the pools on withing the get methods of CoreMessageObjectPools to get the
specific pool, to avoid having this scattered every where.



View raw message