lookfirst/jmxtrans — pull request #45 by kmac (2012-10-12 20:07:53+00:00)
Title: Fix KeyOutWriter to use a single Logger instance per file name 

Files should hopefully be formatted as per project.

* Fix KeyOutWriter to use a single Logger instance per file name
* Refactor KeyOutWriter to act as base class for DailyKeyOutWriter
* Add DailyKeyOutWriter which uses DailyRollingFileAppender

I refactored KeyOutWriter because I wanted to reuse most of the code for DailyKeyOutWriter (if you don't want to take DailyKeyOutWriter then the refactored code will also stand on it's own, but I think it's a useful addition).

Another alternative to having a separate DailyKeyOutWriter would be to support the datePattern property directly in KeyOutWriter, but I thought that might complicate things more than it would having a separate class.

I have some wiki text for DailyKeyOutWriter as well, but I haven't figured out how to merge that into the master wiki yet...

cheers,
Kyle