markap14 commented on issue #3987: NIFI-7015 - ConsumeJMS supports additional message types URL: https://github.com/apache/nifi/pull/3987#issuecomment-574397001 Thanks @tpalfy, this is really nicely done! Was able to test it locally. I tried with TextMessage and BytesMessage, which we originally supported and all worked well. Tried with the new ObjectMessage and StreamMessage and these worked exactly as expected. MapMessage wrote out the JSON as intended. Also tried with an ActiveMQBlobMessage, since I was testing against ActiveMQ. As expected, it did receive the message but failed to process it, so it pushed it back to the error queue. All works exactly as expected. +1 will merge to master. Thanks again!
---------------------------------------------------------------- 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
