Github user michaelandrepearce commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2427#discussion_r233491698
--- Diff:
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/message/impl/CoreMessage.java
---
@@ -1133,8 +1158,7 @@ public Object removeProperty(final SimpleString key) {
@Override
public Object removeProperty(final String key) {
messageChanged();
- checkProperties();
- Object oldValue =
properties.removeProperty(SimpleString.toSimpleString(key,
getPropertyKeysPool()));
+ Object oldValue =
checkProperties().removeProperty(SimpleString.toSimpleString(key,
getPropertyKeysPool()));
--- End diff --
delegate to
public Object removeProperty(final SimpleString key) {
---
