You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@kafka.apache.org by Ewen Cheslack-Postava <me...@ewencp.org> on 2014/10/27 17:55:35 UTC

Review Request 27232: Patch for KAFKA-559

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27232/
-----------------------------------------------------------

Review request for kafka.


Bugs: KAFKA-559
    https://issues.apache.org/jira/browse/KAFKA-559


Repository: kafka


Description
-------

Addressing Joel's comments.


Fix naming: entires -> entries.


Only remove partitions from a group if all partitions were last modified before the threshold date.


Diffs
-----

  core/src/main/scala/kafka/tools/CleanupObsoleteZkEntries.scala PRE-CREATION 

Diff: https://reviews.apache.org/r/27232/diff/


Testing
-------


Thanks,

Ewen Cheslack-Postava