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

    https://github.com/apache/nifi/pull/2458#discussion_r171352969
  
    --- Diff: 
nifi-nar-bundles/nifi-jms-bundle/nifi-jms-processors/src/main/java/org/apache/nifi/jms/processors/ConsumeJMS.java
 ---
    @@ -136,9 +155,16 @@
             relationships = Collections.unmodifiableSet(_relationships);
         }
     
    +    @OnScheduled
    --- End diff --
    
    @mosermw thanks for the explanation! Makes sense. I would recommend 
reconsidering where it is implemented, though. Would look at either creating a 
new Validator in StandardValidators that allows EL without FlowFile attributes 
and evaluates that before validating the result, or otherwise just removing the 
validator all together and implementing it in customValidate... I would just 
prefer to see this done as part of the validation, rather than when the user 
attempts to start the processor. Make sense?


---

Reply via email to