martinzink commented on PR #1504: URL: https://github.com/apache/nifi-minifi-cpp/pull/1504#issuecomment-1424447487
I've followed @szaszm advice and readded ExecuteScript in https://github.com/apache/nifi-minifi-cpp/pull/1504/commits/47c3922477dfbe16ad8eec0ab979e32c717efa5d, so this PR doesn't break compatibility with older configs, and follows NiFi's way of doing ExecuteScript more closely. A ScriptExecutor is loaded through ClassLoader, this means to use ExecuteScript you will need the script-extension aswell as the python-script-extension/lua-script-extension to use python/lua scripting. This intruduces a bit of additional complexity (and maybe restrictions) vs the separate ExecuteLuaScript/ExecutePythonScript approach. Let me know what you think. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
