You are viewing a plain text version of this content. The canonical link for it is here.

Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2021/10/17 00:57:03 UTC

[GitHub] [activemq-artemis] clebertsuconic commented on pull request #3796: ARTEMIS-3461 Generalize MBean Support on Messages and avoid converstion to core on AMQP Messages on console browsing

clebertsuconic commented on pull request #3796:
URL: https://github.com/apache/activemq-artemis/pull/3796#issuecomment-945029293


   @erwindon can you send me your test set somehow? I was actually looking for something that would break the conversion and would pass later.
   
   just send me the raw test and I can convert it into an unit test.
   
   
   on my test, I only have the AMQPProperties.properties.getAddress() and that's showing correctly on the console.
   
   
   on the core case I don't see that either. so I"m not sure what you're referring..
   
   
   if you send me your test set of messages that would break the conversion I will create a test case and merge the whole thing.
   
   
   I'm going to squash my changes now. as commit 13e7c60237abd55485153c6f19e95fee56ad5f85


-- 
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.

To unsubscribe, e-mail: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org