mattyb149 commented on code in PR #10006:
URL: https://github.com/apache/nifi/pull/10006#discussion_r2138845903
##########
nifi-assembly/pom.xml:
##########
@@ -519,6 +519,24 @@ language governing permissions and limitations under the
License. -->
<version>2.5.0-SNAPSHOT</version>
<type>nar</type>
</dependency>
+ <dependency>
+ <groupId>org.apache.nifi</groupId>
+ <artifactId>nifi-cql-nar</artifactId>
Review Comment:
For all external system integrations I'm in favor of a (perhaps set of)
interface(s) that separate the external system's API/SDK from the NiFi
components in order to BYO library so we can try to be future-proof for
upcoming versions and such. This is often a lot more difficult and
time-consuming than when we used to just accept all contributions that
connected to these systems.
Having said that, I don't want to set a negative tone for PR contributions
for external system integrations or else everyone will end up with their own
private repos and NiFi's bread and butter of universal data distribution will
suffer.
--
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]
