{"action":"created","issue":{"url":"https://api.github.com/repos/apache/kafka/issues/4283","repository_url":"https://api.github.com/repos/apache/kafka","labels_url":"https://api.github.com/repos/apache/kafka/issues/4283/labels{/name}","comments_url":"https://api.github.com/repos/apache/kafka/issues/4283/comments","events_url":"https://api.github.com/repos/apache/kafka/issues/4283/events","html_url":"https://github.com/apache/kafka/pull/4283","id":278500721,"number":4283,"title":"KAFKA-6193: Only delete reassign_partitions znode after reassignment is complete","user":{"login":"ijuma","id":24747,"avatar_url":"https://avatars2.githubusercontent.com/u/24747?v=4","gravatar_id":"","url":"https://api.github.com/users/ijuma","html_url":"https://github.com/ijuma","followers_url":"https://api.github.com/users/ijuma/followers","following_url":"https://api.github.com/users/ijuma/following{/other_user}","gists_url":"https://api.github.com/users/ijuma/gists{/gist_id}","starred_url":"https://api.github.com/users/ijuma/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ijuma/subscriptions","organizations_url":"https://api.github.com/users/ijuma/orgs","repos_url":"https://api.github.com/users/ijuma/repos","events_url":"https://api.github.com/users/ijuma/events{/privacy}","received_events_url":"https://api.github.com/users/ijuma/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2017-12-01T15:26:54Z","updated_at":"2017-12-06T02:47:00Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/apache/kafka/pulls/4283","html_url":"https://github.com/apache/kafka/pull/4283","diff_url":"https://github.com/apache/kafka/pull/4283.diff","patch_url":"https://github.com/apache/kafka/pull/4283.patch"},"body":"- Ensure that `partitionsBeingReassigned` is fully populated before\r\n`removePartitionFromReassignedPartitions` is invoked. This is\r\nnecessary to avoid premature deletion of the `reassign_partitions`\r\nznode.\r\n- Modify and add tests to verify the fixes.\r\n- Add documentation.\r\n- Use `info` log message if assignedReplicas == newReplicas and\r\nremove control flow based on exceptions.\r\n- General logging improvements.\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/349514002","html_url":"https://github.com/apache/kafka/pull/4283#issuecomment-349514002","issue_url":"https://api.github.com/repos/apache/kafka/issues/4283","id":349514002,"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-12-06T02:47:00Z","updated_at":"2017-12-06T02:47:00Z","author_association":"CONTRIBUTOR","body":"Saw another failure of this test here: https://builds.apache.org/job/kafka-pr-jdk9-scala2.12/2983/console\r\n\r\nFrom the errors I think it is quite likely due to the issue that it is fixing. Made a pass over the PR and LGTM."}}
{"actor":{"display_login":"guozhangwang"}}