bejancsaba commented on code in PR #6733:
URL: https://github.com/apache/nifi/pull/6733#discussion_r1067432784


##########
c2/c2-client-bundle/c2-client-service/src/main/java/org/apache/nifi/c2/client/service/operation/C2OperationHandler.java:
##########
@@ -48,6 +48,14 @@ public interface C2OperationHandler {
      */
     Map<String, Object> getProperties();
 
+    /**
+     * Determines if the given operation requires to restart the MiNiFi process
+     * @return true if it requires restart, false otherwise
+     */
+    default boolean requiresRestart() {

Review Comment:
   Yeah so by client I mean any implementation of the c2 functionality that is 
not the NiFi one. As you stated this current implementation while we try to 
keep it "clean" makes assumptions about how NiFi works. Anyway this is the only 
implementation / usage at the moment so we can surely live with it just wanted 
to discuss.



-- 
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]

Reply via email to