Github user bbende commented on the issue:

    https://github.com/apache/nifi/pull/834
  
    Performed a couple of test runs and everything is working nicely, thanks 
for making all the updates.
    
    My last comment is that we may need to add something to the toolkit NOTICE 
for Grooby. The main NiFi assembly seems to have this for the scripting 
processors bringing in groovy, only difference being groovy.jar vs. 
groovy-all.jar: 
https://github.com/apache/nifi/blob/master/nifi-assembly/NOTICE#L815
    
    If we add that (or we have a reason we don't need it) then I am a +1 and 
good to merge, nice work!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to