Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/728#issuecomment-188822872 Yes I am aware of it. We are in the process of merging with the JStorm project, and part of this merger involves moving most of the clojure code to java. In all likelihood the degradation is due to reflection happening on the critical path because many new places in the code are calling into java that were not doing so before. I plan on doing some profiling soon and submit some fixes, but it is very much a moving target so I have not been very motivated to do so. The code for the netty messaging layer should stay mostly the same between 1.0 and 2.0 for this, so if you want to do your work based off of the 1.0 branch and show performance comparisons there being the same I would totally accept that as proof that the code is good.  - Bobby On Thursday, February 25, 2016 12:53 AM, Sun <notificati...@github.com> wrote: @revans2 Just merged code from master and seems there is performance degradation with recent changes. I noticed that it not only affects this branch, but also the master branch. Here are the comparison on my laptop for running the perf test with only 5,000/sec throughput with code from master.Before (git hash: 3db9680) netty3-r5000.txtNow (git hash: bd396b3) netty3-new-r5000.txtNote: when running the latest code, my CPU is almost 100% busy which may explain why it was so much worse. When running the "old" code, my CPU still had 10% idle time.-thanksâ Reply to this email directly or view it on GitHub.
--- 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. ---