nifi-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From olegz <...@git.apache.org>
Subject [GitHub] nifi pull request: Eorgad patch nifi 539
Date Mon, 16 May 2016 18:13:56 GMT
Github user olegz commented on the pull request:

    https://github.com/apache/nifi/pull/416#issuecomment-219501349
  
    @eorgad First, thank you for your contribution. This is definitely something people had
asked for. 
    Now, . . . . I see that you added it to the standard processors. What I would suggest
is to make it a separate bundle in 'nifi-nar-bundles'. Basically we are at the point where
we are trying to be cognizant of a NiFi size and do not put anything new into standard processors.
So it has to be a separate bundle. And while we are in the early stages of Extension Registry
discussion, having separate bundles, would still (even today) allow anyone to add/remove what
they don't want in NiFI while with the current approach you've taken it won't be easy to do.

    Let us know if you need more guidance, but I would also suggest to look into any bundle
in 'nifi-nar-bundles' as an example.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message