{"action":"created","issue":{"url":"https://api.github.com/repos/apache/kafka/issues/3603","repository_url":"https://api.github.com/repos/apache/kafka","labels_url":"https://api.github.com/repos/apache/kafka/issues/3603/labels{/name}","comments_url":"https://api.github.com/repos/apache/kafka/issues/3603/comments","events_url":"https://api.github.com/repos/apache/kafka/issues/3603/events","html_url":"https://github.com/apache/kafka/pull/3603","id":246923044,"number":3603,"title":"KAFKA-5671 Followup: Remove reflections in unit test classes","user":{"login":"guozhangwang","id":10729597,"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},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":23,"created_at":"2017-08-01T00:35:51Z","updated_at":"2017-08-02T22:14:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/kafka/pulls/3603","html_url":"https://github.com/apache/kafka/pull/3603","diff_url":"https://github.com/apache/kafka/pull/3603.diff","patch_url":"https://github.com/apache/kafka/pull/3603.patch"},"body":"1. Remove rest deprecation warnings in streams:jar.\r\n\r\n2. Consolidate all unit test classes' reflections to access internal topology builder from packages other than `o.a.k.streams`. We need to refactor the hierarchies of StreamTask, StreamThread and KafkaStreams to remove these hacky reflections.\r\n\r\n3. Minor fixes such as reference path, etc.\r\n\r\n4. Minor edits on web docs for the describe function under developer-guide."},"comment":{"url":"https://api.github.com/repos/apache/kafka/issues/comments/319813873","html_url":"https://github.com/apache/kafka/pull/3603#issuecomment-319813873","issue_url":"https://api.github.com/repos/apache/kafka/issues/3603","id":319813873,"user":{"login":"guozhangwang","id":10729597,"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":"2017-08-02T22:14:07Z","updated_at":"2017-08-02T22:14:07Z","body":"I think it is okay to keep it in the specific test class than creating two more XXTestUtil classes. "}}
{"actor":{"display_login":"guozhangwang"}}