Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/1147#issuecomment-193839237
@unsleepy22 It looks a lot better. I don't care as much about the exact
API. we can change that in the future the important part is removing the
dependency on clojure. I have one minor issue, but overall it looks good and I
am +1 for merging it in.
StatsUtils.java 2045-2046: in weightAvgAndSum
The reason for having the productOr0 function is because the inputs could
be null, and if they were null we didn't want to blow up with an NPE. The
current code will throw an NPE. it should probably look more like.
```
ret += productOr0(entry.getValue(), id2num.get(k));
```
I plan to do a bit of testing myself, but great work.
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
