Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/897#issuecomment-159290747
The latest changes look fine to me I am +1 from a code perspective
We do not currently have performance tests that use multi-lang, but it
would not be hard to modify ThroughputVsLatency to optionally use it. It is
just an optimized version of word count, so replacing the split sentence bolt
with the one from WordCountTopology would be fairly trivial. The biggest issue
would be around CPU usage. The automated reporting only looks at the java
process, and does not include the children. Looking at the SIGAR API it does
not look like there is a good way to get the child processes though. SO you may
need to compare CPU manually for now.
---
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.
---
