turcsanyip commented on a change in pull request #4516:
URL: https://github.com/apache/nifi/pull/4516#discussion_r616081804



##########
File path: 
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/RemoteProcessGroupResource.java
##########
@@ -829,6 +833,105 @@ public Response updateRemoteProcessGroupRunStatus(
         );
     }
 
+    /**
+     * Updates the operational status for all remote process groups in the 
specified process group with the specified value.
+     *
+     * @param httpServletRequest                request
+     * @param processGroupId                    The id of the process group in 
which all remote process groups to update.
+     * @param requestRemotePortRunStatusEntity  A remotePortRunStatusEntity 
that holds the desired run status
+     * @return A response with an array of RemoteProcessGroupEntity objects.
+     */
+    @PUT
+    @Consumes(MediaType.APPLICATION_JSON)
+    @Produces(MediaType.APPLICATION_JSON)
+    @Path("process-group/run-status/{id}")

Review comment:
       The existing endpoint for starting/stopping all processors in a process 
group is:
   `/flow/process-groups/{id}`
   For enabling/disabling controller services in a process group:
   `/flow/process-groups/{id}/controller-services`
   
   Following the same scheme, I think the new endpoint should be:
   `/flow/process-groups/{id}/remote-process-groups`
   




-- 
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:
us...@infra.apache.org


Reply via email to