[ 
https://issues.apache.org/jira/browse/NIFI-1202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15983251#comment-15983251
 ] 

ASF GitHub Bot commented on NIFI-1202:
--------------------------------------

Github user mcgilman commented on the issue:

    https://github.com/apache/nifi/pull/1306
  
    @ijokarumawak Yes, your understanding here is correct. Sorry, I was missing 
what's unique about this case which is that it is an **optional** number. I 
don't believe we have this case elsewhere. Other places where we have numbers, 
they have default values and support being updated but not unconfigured.
    
    I have one thought that may make the DTO/API a little more clear. As is, 
the behavior of setting the fields differ within the same DTO and even amongst 
the same fields depending on the presence of other fields. Since we want the 
batch settings to be treated as a single unit in terms of configuration, what 
are your thoughts introducing a `BatchSettingsDTO` for relaying this 
configuration? For clarity, I'm suggesting the possibility of the 
`RemoteProcessGroupPortDTO` having a `BatchSettingsDTO` which has `batchCount`, 
`batchSize`, and `batchDuration`.
    
    With this approach, in the `RemoteProcessGroupPortDTO` the behavior is 
consistent. If a field is set, attempt to configure it. If a field is null, 
leave the existing configuration in place. Within the `BatchSettingsDTO` we can 
stipulate that these fields are a single unit and the proposed configuration 
will replace the existing configuration. I think this approach would be 
functionally equivalent but may be slightly less confusing.
    
    Thoughts?
    
    Not trying to be super picky here but need to be very conscious of the 
changes as they are part of the public REST API. Just trying to make this as 
clear as possible. If someone directly invokes these endpoints I'd hate if they 
inadvertently cleared the existing desired configuration. 


> Allow user to configure Batch Size for site-to-site
> ---------------------------------------------------
>
>                 Key: NIFI-1202
>                 URL: https://issues.apache.org/jira/browse/NIFI-1202
>             Project: Apache NiFi
>          Issue Type: Improvement
>          Components: Core Framework, Core UI, Documentation & Website
>            Reporter: Mark Payne
>            Assignee: Koji Kawamura
>         Attachments: configure-remote-port-window.png, 
> WIP-added-batch-settings-ui.png
>
>
> Currently, there is no way for a user to specify the batch size that 
> Site-to-Site will use. The framework decides this for you. However, if we 
> want to use the List/Fetch Pattern, it will be helpful to specify a small 
> batch size so that a small number of things that are listed are still well 
> distributed across the cluster.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to