Github user ptgoetz commented on the pull request:
https://github.com/apache/incubator-storm/pull/177#issuecomment-52810515
+1
I was originally on the fence in terms of where additional multi-lang
implementation should live (i.e. do they belong in "external" like the kafka
module), but I'm okay with it as-is. If multi-lang maintenance becomes a
burden, we can always switch to that model later.
If we merge this, we should probably follow up with a heads up to the dev@
and user@ lists that nodejs needs to be installed in order to build.
---
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.
---
