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



##########
File path: 
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/NiFiServiceFacade.java
##########
@@ -1301,6 +1301,14 @@
      */
     StatusHistoryEntity getRemoteProcessGroupStatusHistory(String id);
 
+
+    /**
+     * Verifies that transmission state of all remote process groups within 
the specified process group can be updated.
+     * @param processGroupId The process group in which to verify remote 
process groups
+     * @param shouldTransmit The transmission state to verify for
+     */
+    void verifyUpdateRemoteProcessGroups(String processGroupId, boolean 
shouldTransmit);

Review comment:
       Minor: just to make it more clear  it looks to be a good improvement to 
change processGroupId into containingProcessGroupId (it's easy to to slip take 
it as RPG id as the scope is the RPG)




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


Reply via email to