mcgilman commented on pull request #5247:
URL: https://github.com/apache/nifi/pull/5247#issuecomment-1039496144


   @eduardofontes Thanks for the PR! The process of enabling and disabling 
Controller Services is complex. Currently, the NiFi UI supports enabling and 
disabling a single Controller Service. When enabling the user has the option to 
conditionally enable any dependent Controller Services and starting any 
referencing Processors/Reporting Tasks. When disabling the user does not have 
this option as the user must stop all referencing Processors/Reporting Tasks 
and disable any dependent Controller Services. Since there could be any number 
of affected components, this helps keep the operation as concise as possible, 
helps guarantee its success, and provides the user the necessary insight into 
the operation.
   
   The API that your invoking is actually part of a larger sequence [1] to 
update Variables and Parameters that could be referenced by Controller Services 
and their referencing components across all Nodes in a cluster. The UX for 
those actions show the user the affected components and provides updates as the 
cluster performs the sequence of operations. I think this feature would also 
benefit from a similar UX where the user can see the full scope of their action 
and gets updates while the action completes.
   
   [1] 
https://github.com/apache/nifi/blob/main/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/ProcessGroupResource.java#L827


-- 
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: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to