ASF GitHub Bot commented on NIFI-4224:

Github user mcgilman commented on a diff in the pull request:

    --- Diff: 
    @@ -314,7 +430,7 @@ public Response updateProcessGroup(
                         Authorizable authorizable = 
RequestAction.WRITE, NiFiUserUtils.getNiFiUser());
    -                null,
    +                () -> 
    --- End diff --
    This is verifying that the variables can be updated but they are ignored in 
the corresponding `updateProcessGroup` call below. Since variable updates are 
not supported in this endpoint, can this check be removed?

> Add Variable Registry at Process Group level
> --------------------------------------------
>                 Key: NIFI-4224
>                 URL: https://issues.apache.org/jira/browse/NIFI-4224
>             Project: Apache NiFi
>          Issue Type: New Feature
>          Components: Core Framework
>            Reporter: Mark Payne
>            Assignee: Mark Payne
> Currently, NiFi exposes a variable registry that is configurable by adding 
> the name of a properties file to nifi.properties and then treating the 
> referenced properties file as key/value pairs for the variable registry. 
> This, however, is very limiting, as it provides a global scope for all 
> variables, and it requires a restart of NiFi in order to pick up any updates 
> to the file. We should expose a Process Group-level Variable Registry.

This message was sent by Atlassian JIRA

Reply via email to