{"action":"created","issue":{"url":"https://api.github.com/repos/apache/storm/issues/1387","repository_url":"https://api.github.com/repos/apache/storm","labels_url":"https://api.github.com/repos/apache/storm/issues/1387/labels{/name}","comments_url":"https://api.github.com/repos/apache/storm/issues/1387/comments","events_url":"https://api.github.com/repos/apache/storm/issues/1387/events","html_url":"https://github.com/apache/storm/pull/1387","id":152549653,"number":1387,"title":"STORM-1756: Explicitly null KafkaServer reference in KafkaTestBroker …","user":{"login":"srdo","id":4324588,"avatar_url":"https://avatars.githubusercontent.com/u/4324588?v=3","gravatar_id":"","url":"https://api.github.com/users/srdo","html_url":"https://github.com/srdo","followers_url":"https://api.github.com/users/srdo/followers","following_url":"https://api.github.com/users/srdo/following{/other_user}","gists_url":"https://api.github.com/users/srdo/gists{/gist_id}","starred_url":"https://api.github.com/users/srdo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/srdo/subscriptions","organizations_url":"https://api.github.com/users/srdo/orgs","repos_url":"https://api.github.com/users/srdo/repos","events_url":"https://api.github.com/users/srdo/events{/privacy}","received_events_url":"https://api.github.com/users/srdo/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-05-02T13:10:24Z","updated_at":"2016-05-03T10:19:56Z","closed_at":"2016-05-03T10:19:56Z","pull_request":{"url":"https://api.github.com/repos/apache/storm/pulls/1387","html_url":"https://github.com/apache/storm/pull/1387","diff_url":"https://github.com/apache/storm/pull/1387.diff","patch_url":"https://github.com/apache/storm/pull/1387.patch"},"body":"…to prevent out of memory on large test classes.\r\n\r\nI'm not clear on whether JUnit keeps the reference for the lifetime of the module test, or whether they're released after all tests in a class have run. Either way, a test class with a KafkaTestBroker field and many tests would likely still run out of memory."},"comment":{"url":"https://api.github.com/repos/apache/storm/issues/comments/216488648","html_url":"https://github.com/apache/storm/pull/1387#issuecomment-216488648","issue_url":"https://api.github.com/repos/apache/storm/issues/1387","id":216488648,"user":{"login":"srdo","id":4324588,"avatar_url":"https://avatars.githubusercontent.com/u/4324588?v=3","gravatar_id":"","url":"https://api.github.com/users/srdo","html_url":"https://github.com/srdo","followers_url":"https://api.github.com/users/srdo/followers","following_url":"https://api.github.com/users/srdo/following{/other_user}","gists_url":"https://api.github.com/users/srdo/gists{/gist_id}","starred_url":"https://api.github.com/users/srdo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/srdo/subscriptions","organizations_url":"https://api.github.com/users/srdo/orgs","repos_url":"https://api.github.com/users/srdo/repos","events_url":"https://api.github.com/users/srdo/events{/privacy}","received_events_url":"https://api.github.com/users/srdo/received_events","type":"User","site_admin":false},"created_at":"2016-05-03T10:19:56Z","updated_at":"2016-05-03T10:19:56Z","body":"@HeartSaVioR I saw OOME on one of my branches, because each KafkaServerStartable allocates a 130MB byte array for the log cleaner thread. A heap dump showed that there were 8 of these in memory at the time of the exception. I was going off of http://junit.org/junit4/faq.html#atests_18 to explain it, but I can't reproduce it now. I'll close this. "}}