Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/1051#issuecomment-176204284
@hustfxj, I probably was not clear enough in my review comments that
spawned this JIRA. I was thinking along the lines of @HeartSaVioR. Once the
majority of the code is in java we can review types everywhere. I wanted this
JIRA filed so it is one that we do not forget about. If you can make all of
the tests pass before the full java conversion I am OK with it, but like
@HeartSaVioR said you will need to update the clojure code to go from a long
which is the clojure default to a int explicitly.
---
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.
---
