Github user mgaido91 commented on the issue:

    https://github.com/apache/nifi/pull/2294
  
    thanks @MikeThomsen. Actually I was referring at [this 
comment](https://github.com/apache/nifi/pull/2294#discussion_r160140592). I 
think that is my last comment. Overall it LGTM.
    
    On the batch size, I think that the committers who will have to review this 
before merging can express their opinion. Your opinion makes sense, but it 
involves a consistency issue: we need a tradeoff and I think only committers 
can decide what to do.


---

Reply via email to