[
https://issues.apache.org/jira/browse/STORM-386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14102659#comment-14102659
]
ASF GitHub Bot commented on STORM-386:
--------------------------------------
Github user revans2 commented on the pull request:
https://github.com/apache/incubator-storm/pull/177#issuecomment-52682866
Great to hear that the latest code is in prod now.
I ran into a few problems, but none of them are really your code. The test
didn't fail when I didn't have nodejs installed. but this is because of a bug
in util.clj STORM-461
Also I really hate having two copies of storm.js in there, but we have the
same thing for ruby and python. I files STORM-462 for this too.
The last problem I has was with documentation. We don't indicate anywhere
what needs to be installed to build/run the tests. Could you add a section to
DEVELOPER.md that indicates that you need ruby, python, and nodejs for all of
the tests to pass.
I also assume that you are going to be able to help support this long term?
> Development of multilang protocol in nodejs
> -------------------------------------------
>
> Key: STORM-386
> URL: https://issues.apache.org/jira/browse/STORM-386
> Project: Apache Storm (Incubating)
> Issue Type: New Feature
> Environment: nodejs
> Reporter: Anya Tchernishov
>
> Support nodejs multilang protocol.
> Design considerations:
> - Emit will receive an object (like args and kwargs in python) and a callback
> that is called when task ids list is received.
> self.emit({tuple: [word]}, function(taskIds) {
> self.log(word + ' sent to task ids - ' + taskIds);
> });
> - The following methods will received done method that must be invoked on
> completion (same pattern used by the mocha test framework for async unit
> tests).
> - Internal implementation of emit uses Process.stdout.write without a
> callback since nodejs streams maintain FIFO order (so far, we have not found
> a need for providing a callback).
>
--
This message was sent by Atlassian JIRA
(v6.2#6252)
