Github user mattyb149 commented on the pull request:
https://github.com/apache/nifi/pull/185#issuecomment-175092672
TL;DR Will take a fresh look at this :)
I wanted very badly to have a parent Abstract base class, but the two
processors extend from different parent classes (AbstractProcessor and
AbstractSessionFactoryProcessor). Now that the behavior of ExecuteScript has
changed to let the script handle the session and transfers, perhaps I can
refactor that, at which point I will certainly pull the common code out. I
thought about a Util class too but the methods looked like they'd need large
signatures to get all the dependencies injected.
---
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.
---
