{"action":"created","issue":{"url":"https://api.github.com/repos/apache/storm/issues/122","labels_url":"https://api.github.com/repos/apache/storm/issues/122/labels{/name}","comments_url":"https://api.github.com/repos/apache/storm/issues/122/comments","events_url":"https://api.github.com/repos/apache/storm/issues/122/events","html_url":"https://github.com/apache/storm/pull/122","id":34017423,"number":122,"title":"STORM-354: Testing: allow users to pass TEST-TIMEOUT-MS as param for complete-topology","user":{"login":"aaronlevin","id":1580060,"avatar_url":"https://avatars.githubusercontent.com/u/1580060?v=3","gravatar_id":"","url":"https://api.github.com/users/aaronlevin","html_url":"https://github.com/aaronlevin","followers_url":"https://api.github.com/users/aaronlevin/followers","following_url":"https://api.github.com/users/aaronlevin/following{/other_user}","gists_url":"https://api.github.com/users/aaronlevin/gists{/gist_id}","starred_url":"https://api.github.com/users/aaronlevin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aaronlevin/subscriptions","organizations_url":"https://api.github.com/users/aaronlevin/orgs","repos_url":"https://api.github.com/users/aaronlevin/repos","events_url":"https://api.github.com/users/aaronlevin/events{/privacy}","received_events_url":"https://api.github.com/users/aaronlevin/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2014-05-21T19:47:18Z","updated_at":"2015-01-20T16:50:57Z","closed_at":"2014-07-18T13:30:40Z","pull_request":{"url":"https://api.github.com/repos/apache/storm/pulls/122","html_url":"https://github.com/apache/storm/pull/122","diff_url":"https://github.com/apache/storm/pull/122.diff","patch_url":"https://github.com/apache/storm/pull/122.patch"},"body":"It would be nice if `complete-topology` allowed a user to pass in the default timeout as a parameter (rather than just setting it as `5000ms`). I had a test that kept failing by taking too long. \r\n\r\nThis PR adds functionality without breaking any existing code. Tests pass.\r\n\r\nJIRA ticket: https://issues.apache.org/jira/browse/STORM-354"},"comment":{"url":"https://api.github.com/repos/apache/storm/issues/comments/70688283","html_url":"https://github.com/apache/storm/pull/122#issuecomment-70688283","issue_url":"https://api.github.com/repos/apache/storm/issues/122","id":70688283,"user":{"login":"revans2","id":3441321,"avatar_url":"https://avatars.githubusercontent.com/u/3441321?v=3","gravatar_id":"","url":"https://api.github.com/users/revans2","html_url":"https://github.com/revans2","followers_url":"https://api.github.com/users/revans2/followers","following_url":"https://api.github.com/users/revans2/following{/other_user}","gists_url":"https://api.github.com/users/revans2/gists{/gist_id}","starred_url":"https://api.github.com/users/revans2/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/revans2/subscriptions","organizations_url":"https://api.github.com/users/revans2/orgs","repos_url":"https://api.github.com/users/revans2/repos","events_url":"https://api.github.com/users/revans2/events{/privacy}","received_events_url":"https://api.github.com/users/revans2/received_events","type":"User","site_admin":false},"created_at":"2015-01-20T16:50:57Z","updated_at":"2015-01-20T16:50:57Z","body":"https://github.com/apache/storm/blob/master/storm-core/src/clj/backtype/storm/testing.clj#L190-192\r\n\r\nShows that the environment variable STORM_TEST_TIMEOUT_MS, can be used to set the value globally.  We probably should document this better somewhere."}}