[GitHub] nifi issue #2458: NIFI-2630 Allow PublishJMS to send TextMessages

2018-03-02 Thread markap14
Github user markap14 commented on the issue: https://github.com/apache/nifi/pull/2458 Thanks @mosermw! All looks good. Was able to verify functionality of both TextMessage and BytesMessage with existing queues and new queues via ActiveMQ. +1 merged to master. ---

[GitHub] nifi issue #2458: NIFI-2630 Allow PublishJMS to send TextMessages

2018-02-28 Thread mosermw
Github user mosermw commented on the issue: https://github.com/apache/nifi/pull/2458 @markap14 I added a commit to do character set validation in a property validator instead of OnScheduled. ---

[GitHub] nifi issue #2458: NIFI-2630 Allow PublishJMS to send TextMessages

2018-02-15 Thread mosermw
Github user mosermw commented on the issue: https://github.com/apache/nifi/pull/2458 Added a third commit to improve documentation. ---

[GitHub] nifi issue #2458: NIFI-2630 Allow PublishJMS to send TextMessages

2018-02-12 Thread mosermw
Github user mosermw commented on the issue: https://github.com/apache/nifi/pull/2458 I added a second commit, to make the character set encoding configurable for JMS TextMessages, in both PublishJMS and ConsumeJMS. Thanks @markap14 for pointing this out, because this is a good