kevdoran commented on pull request #4933: URL: https://github.com/apache/nifi/pull/4933#issuecomment-808391298
@gruselglatz - There is no reason this updated minifi coming from the nifi code base should not be compatible with the minifi-c2 server. I took a closer look, and now realize I was mistaking this implementation, which acts as a server for the PullHttpChangeIngestor, with [the implementation on this branch](https://github.com/apache/nifi-minifi/tree/minifi-c2-server/minifi-c2), which was a WIP to update the server based on the aforementioned [C2 Protocol](https://cwiki.apache.org/confluence/display/MINIFI/C2+Design). Given that this updated minifi retains the Change Investor capabilities, I'm all for keeping the corresponding server, which I agree is useful. In the future, I would be in favor of adding a Java client implementation of the [C2 Protocol] in this project to be shared be nifi and minifi, which is compatible with the implememtation in `nifi-minifi-cpp`. We should also add a new C2 server implementation for this protocol which could be shared by nifi, minifi, and minifi-cpp, to align all these different flow runtime engines with the same C2 capabilities. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
