[GitHub] storm pull request: STORM-329 : buffer message in client and recon...

[tedxia]

Github user tedxia commented on the pull request:

    https://github.com/apache/storm/pull/268#issuecomment-66422581
  
    I tested this on our cluster, and the result are below:
    
    Before remove sleep in connect
    
![storm](https://cloud.githubusercontent.com/assets/8066671/5373174/d4364ffc-808d-11e4-8641-8647ca4d3581.png)
    
    After remove sleep in connect
    
![stormx1](https://cloud.githubusercontent.com/assets/8066671/5373187/ed25ac7e-808d-11e4-99ce-b63e0d5ea399.png)
    
    After remove sleep in connect, we got more than 100% speed increase, the 
most important thing is that failed Client will never bind send message to 
other worker anymore;
    
    @clockfly  will you test this on your cluster and give some advise.



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