{"action":"created","issue":{"url":"https://api.github.com/repos/apache/kafka/issues/5120","repository_url":"https://api.github.com/repos/apache/kafka","labels_url":"https://api.github.com/repos/apache/kafka/issues/5120/labels{/name}","comments_url":"https://api.github.com/repos/apache/kafka/issues/5120/comments","events_url":"https://api.github.com/repos/apache/kafka/issues/5120/events","html_url":"https://github.com/apache/kafka/pull/5120","id":328644541,"node_id":"MDExOlB1bGxSZXF1ZXN0MTkyMTMwMDc0","number":5120,"title":"KAFKA-6925: fix parentSensors memory leak (#5108)","user":{"login":"vvcephei","id":832787,"node_id":"MDQ6VXNlcjgzMjc4Nw==","avatar_url":"https://avatars0.githubusercontent.com/u/832787?v=4","gravatar_id":"","url":"https://api.github.com/users/vvcephei","html_url":"https://github.com/vvcephei","followers_url":"https://api.github.com/users/vvcephei/followers","following_url":"https://api.github.com/users/vvcephei/following{/other_user}","gists_url":"https://api.github.com/users/vvcephei/gists{/gist_id}","starred_url":"https://api.github.com/users/vvcephei/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vvcephei/subscriptions","organizations_url":"https://api.github.com/users/vvcephei/orgs","repos_url":"https://api.github.com/users/vvcephei/repos","events_url":"https://api.github.com/users/vvcephei/events{/privacy}","received_events_url":"https://api.github.com/users/vvcephei/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-06-01T20:06:23Z","updated_at":"2018-06-01T21:50:38Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/apache/kafka/pulls/5120","html_url":"https://github.com/apache/kafka/pull/5120","diff_url":"https://github.com/apache/kafka/pull/5120.diff","patch_url":"https://github.com/apache/kafka/pull/5120.patch"},"body":"See also #5108 / 0a7462e  and #5119 .\r\n\r\nPreviously, we failed to remove sensors from the parentSensors map, effectively a memory leak.\r\n\r\nAdd a test to verify that removed sensors get removed from the underlying registry as well as the parentSensors map.\r\n\r\nReviewers: Bill Bejeck <bill@confluent.io>, Guozhang Wang <guozhang@confluent.io>\r\n\r\n### Committer Checklist (excluded from commit message)\r\n- [ ] Verify design and implementation \r\n- [ ] Verify test coverage and CI build status\r\n- [ ] Verify documentation (including upgrade notes)\r\n"},"comment":{"url":"https://api.github.com/repos/apache/kafka/issues/comments/394018727","html_url":"https://github.com/apache/kafka/pull/5120#issuecomment-394018727","issue_url":"https://api.github.com/repos/apache/kafka/issues/5120","id":394018727,"node_id":"MDEyOklzc3VlQ29tbWVudDM5NDAxODcyNw==","user":{"login":"guozhangwang","id":10729597,"node_id":"MDQ6VXNlcjEwNzI5NTk3","avatar_url":"https://avatars0.githubusercontent.com/u/10729597?v=4","gravatar_id":"","url":"https://api.github.com/users/guozhangwang","html_url":"https://github.com/guozhangwang","followers_url":"https://api.github.com/users/guozhangwang/followers","following_url":"https://api.github.com/users/guozhangwang/following{/other_user}","gists_url":"https://api.github.com/users/guozhangwang/gists{/gist_id}","starred_url":"https://api.github.com/users/guozhangwang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/guozhangwang/subscriptions","organizations_url":"https://api.github.com/users/guozhangwang/orgs","repos_url":"https://api.github.com/users/guozhangwang/repos","events_url":"https://api.github.com/users/guozhangwang/events{/privacy}","received_events_url":"https://api.github.com/users/guozhangwang/received_events","type":"User","site_admin":false},"created_at":"2018-06-01T21:50:38Z","updated_at":"2018-06-01T21:50:38Z","author_association":"CONTRIBUTOR","body":"> @guozhangwang , the java 10 build failed with this exception: Could not determine java version from '10.0.1'. . Is it the case that 0.11 doesn't build with java 10?\r\n\r\nYeah I think the java 10 support is only added after that."}}
{"actor":{"display_login":"guozhangwang"}}