Github user kishorvpatil commented on the pull request:
https://github.com/apache/incubator-storm/pull/184#issuecomment-48863840
@HeartSaVioR, Recently, I had to create work around #180 for [THRIFT-1665](
https://issues.apache.org/jira/browse/THRIFT-1665) which was fixed in 0.9.1
missing in 0.7.0. Also while I was working on another #150 and #151, I was not
able to use default thrift 0.7.0 compiler instead have to compile from [storm
specific branch](https://github.com/nathanmarz/thrift/tree/storm) in Nathan's
repo. This branch has many patches already available in latest thrift - except
the hash code function. So I thought it might be good to avoid using older
version to gain from many fixed latest thrift already provides and use default
compiler if possible.
@Jens-G As you suggested, trivial hash function(always returning 0) could
be used if that's generic implementation thrift should provide. I suppose, for
storm in particular, since these objects would added to clojure and jvm
collections, performance would be an issue. Since Nathan originally implemented
the thrift patch https://github.com/apache/thrift/pull/156, he might be able to
better justify the change.
---
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.
---
