markap14 commented on PR #11543: URL: https://github.com/apache/nifi/pull/11543#issuecomment-5361057564
Hey @bobpaulin reviewing this again I'm a bit concerned about the direction here. This adds quite a bit of complexity, it means that any components that have a connector method called cannot makes use of any member variables (or rather they can use them but as soon as they get set the object will be destroyed so it doesn't last), and it means that we have very different paths through the code when evaluating a component in the Working Context vs. the Active Context. I'd actually recommend we tackle this issue in a different way, by ensuring that we are always reloading the components as necessary based on classpath changes. Please check this PR that I'd propose as an alternative: https://github.com/apache/nifi/pull/11574 -- 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]
