Github user Parth-Brahmbhatt commented on the pull request:

    https://github.com/apache/storm/pull/303#issuecomment-60635672
  
    search and replace is not a good solution, this seems to be another 
encoding decoding issue. I tested locally and from the initial investigation it 
feels like we have out of sync encoding and decoding schemes. 
    
    word count -> encoded by -> 
https://github.com/apache/storm/blob/master/storm-core/src/clj/backtype/storm/util.clj#L808
 -> results in word+count.
    
    however the ring gives us decoded id at 
https://github.com/apache/storm/blob/master/storm-core/src/clj/backtype/storm/ui/core.clj#L852
 which does not transalte "word+count" to "word count". 
    
    We need to ensure our encoding and decoding is done with the same scheme.


---
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.
---

Reply via email to