Repository: nifi
Updated Branches:
  refs/heads/master 7b5bf265a -> 9de73fbe1


NIFI-4388: Modules Not Honored

The bug is not reproducible in Jython (it handles modules with every script 
reload). But Groovy loads JARs and dirs with classes only on setup.

Signed-off-by: Matthew Burgess <[email protected]>

This closes #2584


Project: http://git-wip-us.apache.org/repos/asf/nifi/repo
Commit: http://git-wip-us.apache.org/repos/asf/nifi/commit/9de73fbe
Tree: http://git-wip-us.apache.org/repos/asf/nifi/tree/9de73fbe
Diff: http://git-wip-us.apache.org/repos/asf/nifi/diff/9de73fbe

Branch: refs/heads/master
Commit: 9de73fbe1f1a7ab226d138732ece8c6e9a0c0fba
Parents: 7b5bf26
Author: Ed <[email protected]>
Authored: Tue Mar 27 00:22:12 2018 -0400
Committer: Matthew Burgess <[email protected]>
Committed: Tue Mar 27 11:20:40 2018 -0400

----------------------------------------------------------------------
 .../apache/nifi/processors/script/InvokeScriptedProcessor.java  | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/nifi/blob/9de73fbe/nifi-nar-bundles/nifi-scripting-bundle/nifi-scripting-processors/src/main/java/org/apache/nifi/processors/script/InvokeScriptedProcessor.java
----------------------------------------------------------------------
diff --git 
a/nifi-nar-bundles/nifi-scripting-bundle/nifi-scripting-processors/src/main/java/org/apache/nifi/processors/script/InvokeScriptedProcessor.java
 
b/nifi-nar-bundles/nifi-scripting-bundle/nifi-scripting-processors/src/main/java/org/apache/nifi/processors/script/InvokeScriptedProcessor.java
index 06a042e..032be32 100644
--- 
a/nifi-nar-bundles/nifi-scripting-bundle/nifi-scripting-processors/src/main/java/org/apache/nifi/processors/script/InvokeScriptedProcessor.java
+++ 
b/nifi-nar-bundles/nifi-scripting-bundle/nifi-scripting-processors/src/main/java/org/apache/nifi/processors/script/InvokeScriptedProcessor.java
@@ -245,10 +245,7 @@ public class InvokeScriptedProcessor extends 
AbstractSessionFactoryProcessor {
                 || ScriptingComponentUtils.MODULES.equals(descriptor)
                 || scriptingComponentHelper.SCRIPT_ENGINE.equals(descriptor)) {
             scriptNeedsReload.set(true);
-            // Need to reset scriptEngine if the value has changed
-            if (scriptingComponentHelper.SCRIPT_ENGINE.equals(descriptor)) {
-                scriptEngine = null;
-            }
+            scriptEngine = null; //reset engine. This happens only when a 
processor is stopped, so there won't be any performance impact in run-time.
         } else if (instance != null) {
             // If the script provides a Processor, call its 
onPropertyModified() method
             try {

Reply via email to