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