On 2011-09-19, Dominik Psenner wrote:

>> 3) - Any date portion in a file name be prefixed with a . as if it were
>> an extension.

> +1, but one should be able to change the "spacer" character. "." is a
> nuisance to regex parse a logfile, where "-" is tangentially easier.

+1 for an option to replace the separator.

>> 3) - Decide if PreserveLogFileNameExtension applies to both the
>> datePattern and the size roll index or just to the size roll index.  I
>> propose that it applies to size roll index and a new parameter
>> (PreserveLogFileBase - defaults to true) be used to apply to the date
>> roll.

> *hmm* Could we make up an easier configuration as we're touching it already?
> Just like in 4), I would love to see all rolling configuration stuff in a
> section named <rollFileConfiguration> that lives near <datePattern>. I.e.
> something like this:

> <file value="bla.log"/>
> <datePattern value="yyyyMMddHHmm" />
> <rollFileConfiguration>
> <rollFileCondition size="5MB" />
> <rollFileLimitation maxcount="5" />
> </rollFileConfiguration>

Grouping the properties that affect the rolling strategy and separating
them from the others makes sense to me.

Stefan

Reply via email to