Github user revans2 commented on the pull request:

    https://github.com/apache/storm/pull/870#issuecomment-155579912
  
    OK Don't sell your idea short just yet.  I did some tuning of the 
ExecutorService and the numbers are on par with prior to batching.
    
    user:      4,552
    sys:      5,015
    
    These are just the CPU ms over a 30 second interval, so smaller is better 
here.  I will check in the changes so you can have a look.  There is still some 
code cleanup to do and more testing before I consider it ready for a full pull 
request but it is looking very promising.


---
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