Github user mosermw commented on the issue:

    https://github.com/apache/nifi/pull/1071
  
    For backwards compatibility purposes, I think changing the default 100 
Maximum Number of Bins to 5 will have a large impact on users.  Right now most 
people just leave this at the default.  If they upgrade NiFi and they were 
regularly using 10 bins, for instance, then after the upgrade their 
MergeContent will behave a LOT differently.
    
    What memory footprint are we targetting for this change?  Even with a 512 
MB heap, 100 bins does not cause problems.
    
    I have really big concerns about this one.  Can we discuss more?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to