exceptionfactory commented on pull request #5692: URL: https://github.com/apache/nifi/pull/5692#issuecomment-1020481969
There is a good deal of code duplication in certain places right now, so this is not necessarily so different, but it is an opportunity to find a better approach. The idea of a `nifi-dbcp-service-meta` sounds a good potential solution. I agree this does not follow the usual pattern of a utility module, so other approaches are certainly worth considering. Moving the META-INF/services definition to a separate module would allow a `nifi-snowflake-service` module to depend on `nifi-dbcp-service`, so that sounds like a workable solution. -- 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
