GitHub user ijuma opened a pull request: https://github.com/apache/kafka/pull/4143
MINOR: PartitionReassignmentHandler should generate event only when znode is created and eliminate unnecessary allocations As suggested by Jun in https://github.com/apache/kafka/pull/4141, we only need to generate the event when the znode is created ( controller failover restarts the reassignment without generating an event). I also took the chance to eliminate all the unnecessary allocations of `TopicPartition` and `TopicAndPartition`. We now use the former in the Controller (bringing it inline with the rest of the non legacy code). You can merge this pull request into a Git repository by running: $ git pull https://github.com/ijuma/kafka partition-reassignment-ignore-handle-deletion-and-data-change Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/4143.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #4143 ---- commit f2a81b5ef493e428d93be3340148f101089d87e3 Author: Ismael Juma <ism...@juma.me.uk> Date: 2017-10-27T09:46:02Z Don't create PartitionReassignment event if the znode is deleted or changed commit 9f448fda833db54e2d558fc69f0e882f9262fab6 Author: Ismael Juma <ism...@juma.me.uk> Date: 2017-10-27T10:25:34Z Use TopicPartition in Controller and avoid allocations ---- ---