Github user revans2 commented on the issue: https://github.com/apache/storm/pull/2591 This works and I am OK with merging it in +1. But the implementation of deserializing the hooks twice feels really odd and non-intuitive to me. But this is not your problem so it should be fine. Please do put up a pull request for the master branch too. I think this should probably apply to the other 1.x branches without any changes.
---