Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/728#issuecomment-170145276
I just ran some performance tests using
https://github.com/apache/storm/blob/master/examples/storm-starter/src/jvm/storm/starter/ThroughputVsLatency.java
I ran with 4 workers on a MacBookPro and the numbers don't look good for
netty4.
With the older code I was able to do 20,000 sentences/second at
99%-ile: 43 ms
99.9%-ile: 99 ms
min: 8.8 ms
max: 177 ms
mean: 16 ms
CPU Utilization over 30 second interval:
user: 159,718 ms
sys: 49,670 ms
But with the netty4 patch it could only handle 17,000/sec and the latency
was much worse
99%-lie: 148 ms
99.9%-lie: 230 ms
min: 9.2 ms
max: 332 ms
mean: 32 ms
CPU:
user: 162,984 ms
sys: 55,867 ms
To have the netty4 patch have similar latency we needed to only run at
14,000 sentences per second.
99%-lie: 38 ms
99.9%-lie: 55 ms
min: 8.8 ms
max: 76 ms
mean: 16 ms
CPU:
user: 152,865 ms
sys: 52,535 ms
Unless we can get the numbers to be close to or better than the netty3
implementation I cannot let this in.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
