clebertsuconic commented on a change in pull request #2695: ARTEMIS-2372 
Filtering on Message Annotations
URL: https://github.com/apache/activemq-artemis/pull/2695#discussion_r291176728
 
 

 ##########
 File path: 
artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/broker/AMQPMessage.java
 ##########
 @@ -1263,7 +1263,10 @@ public Long getLongProperty(String key) throws 
ActiveMQPropertyConversionExcepti
 
    @Override
    public Object getObjectProperty(String key) {
-      if (key.equals(MessageUtil.TYPE_HEADER_NAME.toString())) {
+
+      if 
(key.startsWith(AMQPMessageSupport.JMS_AMQP_MESSAGE_ANNOTATION_PREFIX)) {
 
 Review comment:
   Ok, i will use extraProperties and do something about that...
   
   Just so you know this PR was about exposing the properties.. the use of 
messageAnnotations for this wasn't introduced here.
   
   
   But i will fix it as part of this PR.
   
   
   i will close it for now as I won't have time to finish it before 1 week. I 
may reopen it later.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to