[GitHub] storm issue #2325: [STORM-2738] The number of ackers should default to the n...

[Ethanlm]

Github user Ethanlm commented on the issue:

    https://github.com/apache/storm/pull/2325
  
    Two tests:
    **Confg**
    `topology.workers` not set;
    `topology.acker.executors` not set.
    `topology.component.resources.onheap.memory.mb` is default: 128.
    `worker.heap.memory.mb` is default: 768.
    
    **Test1**
    
![image](https://user-images.githubusercontent.com/14900612/30454031-80ca13be-9960-11e7-8f8d-844310b5f97d.png)
    numOfAckerExec = Math.ceil(12 * 128 / 768) = 2. 
    
    **Test2**
    
    
![image](https://user-images.githubusercontent.com/14900612/30454366-ad82b9b4-9961-11e7-8085-7556968ffb3a.png)
    
    numOfAckerExec =Math.ceil(25 * 128 / 768) = 5. 



---