GitHub user franz1981 opened a pull request:
https://github.com/apache/activemq-artemis/pull/2427
ARTEMIS-2170 Optimized CoreMessage's checkProperties and
cleanupInternalProperties methods
It includes 2 optimizations related to CoreMessage hot paths operations:
checkProperties and cleanupInternalProperties.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/franz1981/activemq-artemis ARTEMIS-2170
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/activemq-artemis/pull/2427.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 #2427
----
commit f4f1e125603f6c510541395a102df68bb7dae66e
Author: Francesco Nigro <nigro.fra@...>
Date: 2018-11-11T09:44:29Z
ARTEMIS-2170 Optimized CoreMessage's cleanupInternalProperties method
The cleanup is now performed into TypedProperties both
for performance reasons (avoilock/unlock many times)
and consistency, given that the operation is now atomic.
commit 8b6e9a417c39ef6b043b7b11c504254e6721d10b
Author: Francesco Nigro <nigro.fra@...>
Date: 2018-11-11T10:32:49Z
ARTEMIS-2170 Optimized CoreMessage's checkProperties method
Any checkProperties();<usage of this.properties> pattern has been
replaced by an atomic checkProperties().<usage of returned properties>
to help both performance and consistency.
----
---
