[GitHub] storm pull request: [STORM-1707] Remove two minute timeout after w...

2016-05-17 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/1370#issuecomment-219854927 Pending #1427 (1.x) to be verified. --- 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

[GitHub] storm pull request: [STORM-1707] Remove two minute timeout after w...

2016-05-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/1370#issuecomment-219622509 @ppoulosk No problem, please take your time and just ping me once it's ready. Thanks in advance! --- If your project is set up for it, you can reply to this email

[GitHub] storm pull request: [STORM-1707] Remove two minute timeout after w...

2016-05-16 Thread ppoulosk
Github user ppoulosk commented on the pull request: https://github.com/apache/storm/pull/1370#issuecomment-219431285 My apologies. I've been juggling non-storm tasks with storm tasks and this dropped in my priority queue. I'll make time to get this done in the next two days. ---

[GitHub] storm pull request: [STORM-1707] Remove two minute timeout after w...

2016-05-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/1370#issuecomment-219390639 @ppoulosk Any updates on this? I couldn't cherry-pick since it modifies ported files. --- If your project is set up for it, you can reply to this email and have

[GitHub] storm pull request: [STORM-1707] Remove two minute timeout after w...

2016-05-03 Thread ppoulosk
Github user ppoulosk commented on the pull request: https://github.com/apache/storm/pull/1370#issuecomment-216527257 Will do. Thanks. --- 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

[GitHub] storm pull request: [STORM-1707] Remove two minute timeout after w...

2016-05-03 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/1370#issuecomment-216472136 +1 Nice. @ppoulosk Could you craft pull requests against 1.x-branch, too? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] storm pull request: [STORM-1707] Remove two minute timeout after w...

2016-05-03 Thread hustfxj
Github user hustfxj commented on the pull request: https://github.com/apache/storm/pull/1370#issuecomment-216446889 LGTM --- 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

[GitHub] storm pull request: [STORM-1707] Remove two minute timeout after w...

2016-04-29 Thread jerrypeng
Github user jerrypeng commented on the pull request: https://github.com/apache/storm/pull/1370#issuecomment-215826814 +1 --- 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

[GitHub] storm pull request: [STORM-1707] Remove two minute timeout after w...

2016-04-29 Thread redsanket
Github user redsanket commented on the pull request: https://github.com/apache/storm/pull/1370#issuecomment-215741544 LGTM +1 --- 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

[GitHub] storm pull request: [STORM-1707] Remove two minute timeout after w...

2016-04-27 Thread ppoulosk
GitHub user ppoulosk opened a pull request: https://github.com/apache/storm/pull/1370 [STORM-1707] Remove two minute timeout after worker launch This is a logic change the removes the two minute wait after worker launch in sync processes. Instead of sitting in a wait loop for all