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

(Updated Oct. 31, 2013, 10:22 p.m.)


Review request for kafka and Jay Kreps.


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


Repository: kafka


Description (updated)
-------

Removed an unncessary log statement and fixed a few more tests that initially 
had the Log public API change


Per Jay's suggestion, avoid changing the public API of Log


Avoid printing clean shutdown file log message on every log


KAFKA-1107 Broker unnecessarily recovers all logs when upgrading from 0.8 to 
0.8.1; Fix includes modifying Log to avoid recovery if the clean shutdown file 
exists; LogManager deletes the clean shutdown file after all logs in a log 
directory have finished loading; If the clean shutdown file does not exist, 
fall back to 0.8.1 recovery logic


Diffs (updated)
-----

  core/src/main/scala/kafka/log/Log.scala 
0cc402b13e8484ae5569f1b8ff7156331a2f82d7 
  core/src/main/scala/kafka/log/LogManager.scala 
d489e08452ab97334d504f76f381eb314ec56901 
  core/src/test/scala/unit/kafka/log/LogTest.scala 
140317c6ab6741308d125e9c1f43078b672c5f95 

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


Testing
-------


Thanks,

Neha Narkhede

Reply via email to