{"action":"created","issue":{"url":"https://api.github.com/repos/apache/storm/issues/429","labels_url":"https://api.github.com/repos/apache/storm/issues/429/labels{/name}","comments_url":"https://api.github.com/repos/apache/storm/issues/429/comments","events_url":"https://api.github.com/repos/apache/storm/issues/429/events","html_url":"https://github.com/apache/storm/pull/429","id":57360020,"number":429,"title":"STORM-329: fix cascading Storm failure by improving reconnection strategy and buffering messages","user":{"login":"miguno","id":294849,"avatar_url":"https://avatars.githubusercontent.com/u/294849?v=3","gravatar_id":"","url":"https://api.github.com/users/miguno","html_url":"https://github.com/miguno","followers_url":"https://api.github.com/users/miguno/followers","following_url":"https://api.github.com/users/miguno/following{/other_user}","gists_url":"https://api.github.com/users/miguno/gists{/gist_id}","starred_url":"https://api.github.com/users/miguno/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/miguno/subscriptions","organizations_url":"https://api.github.com/users/miguno/orgs","repos_url":"https://api.github.com/users/miguno/repos","events_url":"https://api.github.com/users/miguno/events{/privacy}","received_events_url":"https://api.github.com/users/miguno/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":35,"created_at":"2015-02-11T18:59:03Z","updated_at":"2015-02-25T00:13:11Z","closed_at":"2015-02-23T21:11:39Z","pull_request":{"url":"https://api.github.com/repos/apache/storm/pulls/429","html_url":"https://github.com/apache/storm/pull/429","diff_url":"https://github.com/apache/storm/pull/429.diff","patch_url":"https://github.com/apache/storm/pull/429.patch"},"body":"**This PR contains the same code as https://github.com/apache/storm/pull/428 but as a single commit for a cleaner commit history of our Storm repo.**\r\n\r\n--\r\n\r\nThis is an improved version of the original pull request discussed at https://github.com/apache/storm/pull/268.  Please refer to the discussion in the link above.\r\n\r\n**Note**:  Please give attribution to @tedxia when merging the pull request as he contributed a lot to this work.\r\n\r\nThe changes of this pull request include:\r\n\r\n- Most importantly, we fix a bug in Storm that may cause a cascading failure in a Storm cluster, to the point where the whole cluster becomes unusable.  This is achieved by the work described in the next bullet points.\r\n- We refactor and improve the Netty messaging backend, notably the client.\r\n- During the initial startup of a topology, Storm will now wait until worker (Netty) connections are ready for operation.  See the [original discussion thread](https://github.com/apache/storm/pull/268) for the detailed explanation and justification of this change.\r\n\r\n@clockfly, @tedxia: Please add any further comments to STORM-329 to this pull request, if possible."},"comment":{"url":"https://api.github.com/repos/apache/storm/issues/comments/75879629","html_url":"https://github.com/apache/storm/pull/429#issuecomment-75879629","issue_url":"https://api.github.com/repos/apache/storm/issues/429","id":75879629,"user":{"login":"ptgoetz","id":260896,"avatar_url":"https://avatars.githubusercontent.com/u/260896?v=3","gravatar_id":"","url":"https://api.github.com/users/ptgoetz","html_url":"https://github.com/ptgoetz","followers_url":"https://api.github.com/users/ptgoetz/followers","following_url":"https://api.github.com/users/ptgoetz/following{/other_user}","gists_url":"https://api.github.com/users/ptgoetz/gists{/gist_id}","starred_url":"https://api.github.com/users/ptgoetz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ptgoetz/subscriptions","organizations_url":"https://api.github.com/users/ptgoetz/orgs","repos_url":"https://api.github.com/users/ptgoetz/repos","events_url":"https://api.github.com/users/ptgoetz/events{/privacy}","received_events_url":"https://api.github.com/users/ptgoetz/received_events","type":"User","site_admin":false},"created_at":"2015-02-25T00:13:11Z","updated_at":"2015-02-25T00:13:11Z","body":"@danielschonfeld No worries. There will be a 0.9.4 release that includes this fix. No need to wait for 0.10.0."}}