Two patches needing review

[Jay Kreps <ja...@gmail.com>]

This is a set of good housekeeping things for 0.8 open source release:
https://issues.apache.org/jira/browse/KAFKA-718
This is just a small set of trivial cleanups that are very visible:
- Fix logging for tools so we don't spew log messages in with tool output
- Fix GC settings so we don't default to "stop the world"
- Modularlize the script variables so you can override just gc, logging, etc
- Remove delete-topic.sh because it doesn't work.
- Remove incorrect things from README
- Remove unused properties and deprecated properties from
config/server.properties that give warnings
- Misc. cleanup of config comments that were incorrect

This is the fsync management patch for trunk:
https://issues.apache.org/jira/browse/KAFKA-615
I can't really do any testing on trunk until this is in.

Por favor.

-Jay

Re: Two patches needing review

[Jun Rao <ju...@gmail.com>]

Sorry for the delay. Reviewed both.

Jun


On Sat, Aug 3, 2013 at 10:52 AM, Jay Kreps <ja...@gmail.com> wrote:

> This is a set of good housekeeping things for 0.8 open source release:
> https://issues.apache.org/jira/browse/KAFKA-718
> This is just a small set of trivial cleanups that are very visible:
> - Fix logging for tools so we don't spew log messages in with tool output
> - Fix GC settings so we don't default to "stop the world"
> - Modularlize the script variables so you can override just gc, logging,
> etc
> - Remove delete-topic.sh because it doesn't work.
> - Remove incorrect things from README
> - Remove unused properties and deprecated properties from
> config/server.properties that give warnings
> - Misc. cleanup of config comments that were incorrect
>
> This is the fsync management patch for trunk:
> https://issues.apache.org/jira/browse/KAFKA-615
> I can't really do any testing on trunk until this is in.
>
> Por favor.
>
> -Jay
>