Github user srdo commented on the issue: https://github.com/apache/storm/pull/2800 @danny0405 That doesn't sound safe to me. I think you're right that it works fine most of the time, but if there are key collisions or an insert leads the map to get resized, I would think that two threads modifying the map at the same time could interfere with each other. Either way, if you're okay with making the whole function thread safe, I think we should do it.
---