Github user markobean commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/3053#discussion_r223751456
  
    --- Diff: 
nifi-nar-bundles/nifi-jms-bundle/nifi-jms-processors/src/main/java/org/apache/nifi/jms/processors/JMSPublisher.java
 ---
    @@ -89,11 +89,14 @@ void setMessageHeaderAndProperties(final Message 
message, final Map<String, Stri
                 for (Entry<String, String> entry : 
flowFileAttributesToSend.entrySet()) {
                     try {
                         if (entry.getKey().equals(JmsHeaders.DELIVERY_MODE)) {
    -                        
message.setJMSDeliveryMode(Integer.parseInt(entry.getValue()));
    +                        this.jmsTemplate.setExplicitQosEnabled(true);
    --- End diff --
    
    No, it should not. The only changes is setting these values through the 
JmsTemplate vice the Message itself due to stated limitation of the API. The 
net effect is the same.


---

Reply via email to