Github user itaifrenkel commented on the pull request:
https://github.com/apache/storm/pull/286#issuecomment-62295232
@HeartSaVioR @clockfly I think we need to keep the multilang protocl
implementation as simple as possible. A full roundtrip of heartbeat messages is
not that bad, as long as it does not add too much latency. If you would like an
optimization for the rountrip messages then you could consider any emit as an
heartbeat, and trigger the heartbeat rountrip only if there are not enough
emits from the bolt. It makes the java code more complicated :(, but achieves
similar goals, and leaves the multilang implementation simpler :). All-in-all I
think this commit is good, and we could discuss various optimizations later on.
---
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.
---
