Github user HeartSaVioR commented on the pull request:
https://github.com/apache/storm/pull/1217#issuecomment-201709299
I guess @arunmahadevan removes not only instantiation of Long but also
Random.randInt() which makes sense to me.
@roshannaik @arunmahadevan
Btw, please change your BasicTopology to here,
- Roshan's: https://gist.github.com/HeartSaVioR/9fd277307d4d5efcd47f
- Arun's: https://gist.github.com/HeartSaVioR/0e2555633a5f7d12cb68
I pasted functionality about printing metrics from cluster information
periodically. (It came from FastWordCountTopology.)
Why this change is necessary is because I saw the behavior that throughput
of topology from Roshan isn't be consistent even after 10 mins (it's
increasing), so when exactly you refreshed UI can change your numbers. (Please
note that 10m is changed, too)
You may want to increase your benchmark period long enough (by modifying
loop count in main) to see when its speed becomes stabilized. You may also want
to make period as argument.
---
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.
---
