Github user revans2 commented on the pull request:

    https://github.com/apache/storm/pull/354#issuecomment-85509268
  
    @longdafeng I am glad to see the JStorm community starting to help out 
here.  I respect your experience with this, especially since this is your third 
iteration on the code.  ZK load is a big issue and concern for me so I am all 
for trying to adopt what JStorm has done in the area of HA.  Because we are 
working on a separate feature branch for this, where we can break APIs if need 
be, perhaps we can check in the code as it is now, and file a follow on JIRA to 
adopt the model, configs, and ideally some of the code that JStorm is using.  
This would also make combining the two simpler in the future.
    
    @Parth-Brahmbhatt @ptgoetz do either of you have an opinion on this?


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

Reply via email to