exceptionfactory commented on PR #11729: URL: https://github.com/apache/nifi/pull/11729#issuecomment-5878226514
> > With that background, however, I don't agree with this approach and I think a more holistic strategy is needed. > > I'm not really tracking why. Could you elaborate? The Python angle came from seeing a bunch of Python developers using this strategy to call out to CPython in the 1.X era and realizing that adding a feature like this would both insulate them and give other types of developers a richer interface that is still simple to reason about. My primary concern is adding some specific (file-based JSON attribute passing) to a more general Processor. `ExecuteStreamCommand` is common enough as a way to invoke external scripts or commands, including Python, but this goes in a more narrow direction. If the goal is to provide something targeted to Python invocation, then something more specific like `ExecuteExternalPythonCommand` might be warranted. In that case, it would require some more careful design thought to define the contract. For example, FlowFile attributes could be passed as an environment variable directly, not as a file reference. Hopefully that explains more of my concerns with this current proposal. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
